-
Notifications
You must be signed in to change notification settings - Fork 50
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
only run dev with newest python version #1165
Conversation
In this case we are not using the perses/.github/workflows/CI.yaml Line 55 in c7971db
|
I just double checked that this is indeed the case for example in https://github.com/choderalab/perses/actions/runs/4295333336/jobs/7485595343#step:7:236 |
Oh, but according to documentation using |
I think I fixed it, the problem (I think) was the lack of telling micromamba to reinstall if the package is in the env. It would be faster and better to use this kind of syntax to install openmm:
but that would then crash on "dev" but would work for the other openmm version strings... but I have an idea! |
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.
This looks good, thanks!
I think eventually we do want to think about not having to manually change the openmm dev version every time there's a new build.
Description
To save some compute time/carbon, we are only going to test
dev
with the latest python version.Motivation and context
Resolves #???
How has this been tested?
Change log