-
Notifications
You must be signed in to change notification settings - Fork 2
Feature/pixi #39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/pixi #39
Conversation
…I to only run pipeline tests
|
@abradley60 -- can you try running some of your AWS code with the pixi environment? All tests are passing using pixi, but I'm aware we don't have tests for everything right now. I've tried the download-etad function, and that's still working, but haven't tried anything else. |
environment.yml
Outdated
| @@ -1,4 +1,4 @@ | |||
| name: sar-pipeline | |||
| name: default | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this meant to be changed from sar-pipeline?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I must have done that by mistake when trying to work out the pixi export. Will revert
Yep will do |
|
General Question @caitlinadams does the pixi build rely solely on the dependancies in the edit - the project.toml references the appropriate conda env file? |
That's my understanding. Pixi refer to the pixi releated code in the Whenever you run a command with pixi, it will double check that your environment is up-to-date with the manifest (pyproject.toml), update the lock file if the manifest has changed, and then build the environment (see pixi docs.
The pyproject.toml file doesn't actually interact with the conda env file at all. The way I built our pixi environment was:
For all future development, the way we'd use pixi is:
Does that help clarify? |
Yep that makes sense thanks for the explanation. Given the exported |
I've just tested this -- if we explicitly list the versions in the pyproject.toml file, they will come through into the As such, there is an extra step of manually reviewing the pyproject.toml file after running
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks for making those additions. I've put a few suggestions -- but mostly to remove upper bounds I forgot about!
Co-authored-by: Caitlin Adams <[email protected]>
Co-authored-by: Caitlin Adams <[email protected]>
Co-authored-by: Caitlin Adams <[email protected]>
Co-authored-by: Caitlin Adams <[email protected]>
Co-authored-by: Caitlin Adams <[email protected]>
|
@caitlinadams any idea on the test failure? |
|
Looking at the error and the GitHub action definition, I wonder whether it's getting a mismatch between the lockfile and the pyroject.toml. The action seems to want to run So, it should be a matter of pulling the change to the manifest, running |
This adds support for using the pixi package manager, including: