Skip to content

Conversation

@NimaSarajpoor
Copy link

No description provided.

@gitnotebooks
Copy link

gitnotebooks bot commented Jan 29, 2026

Review these changes at https://app.gitnotebooks.com/stumpy-dev/automate/pull/7

@NimaSarajpoor NimaSarajpoor force-pushed the stumpy_actions_with_pixi branch from 0976d65 to f869860 Compare January 29, 2026 07:54
@NimaSarajpoor NimaSarajpoor force-pushed the stumpy_actions_with_pixi branch from 412afe3 to b0c848d Compare January 29, 2026 08:04
@NimaSarajpoor
Copy link
Author

@seanlaw
Apologies for the noise!

I've generated API Key from my account in prefix.dev and saved it in a secret, and used it in the yml file. See Github Actions Run in the forked repo.

@seanlaw
Copy link
Contributor

seanlaw commented Jan 29, 2026

@NimaSarajpoor When I go to:

Settings > Secrets and Variables > Actions

It states:

Anyone with collaborator access to this repository can use these secrets and variables for actions. They are not passed to workflows that are triggered by a pull request from a fork.

So, I believe that the workflow needs to be merged first before it can have access to the token

@NimaSarajpoor
Copy link
Author

@seanlaw

They are not passed to workflows that are triggered by a pull request from a fork.

So, I believe that the workflow needs to be merged first before it can have access to the token

Makes sense! Thanks for checking that out.

@NimaSarajpoor NimaSarajpoor changed the title Install dependencies via pixi and run the tests Check pyfftw Feb 1, 2026
@NimaSarajpoor NimaSarajpoor changed the title Check pyfftw Add workflow to check pyfftw Feb 1, 2026
@NimaSarajpoor
Copy link
Author

NimaSarajpoor commented Feb 1, 2026

@seanlaw
Okay... I think I should finish the sdp PR in STUMPY (stumpy-dev/stumpy#1118) first, and then gets successful run there (without checking pyfftw if it is not installed).... and then, after merging that PR, come back here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants