-
Notifications
You must be signed in to change notification settings - Fork 187
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
failing builds on python 3.7 #4256
Comments
I would say 1 is just as much a hack as number 2. I was going to propose to just switch from Python 3.7 to 3.8 entirely, as we did for the CI workflow, but then I saw that we actually run the I still have some questions though: #. You say the problem is with uninstalling Finally, just wanted to mention that 3 might be even more work than one would think. It is not just |
Correct. I did test this on github actions - uninstalling pyyaml works; installing pyyaml 5.1.2 afterwards works; only directly installing 5.1.2 pyyaml fails.
That is indeed something left to do - I wanted to test this through conda, but python 3.7.8 was not yet available.
It doesn't defeat the purpose - pyyaml is not part of the standard library and won't be installed in a new environment by default. Anyhow, figuring out whether this is a problem of the github actions runner environment or whether it is a problem of pyyaml would be worthwhile (in the first case, we should reopen actions/runner-images#1202 ) |
What if a user tries to install |
I tried to install aiida-core on the official Python docker container and that seems to work without any issue:
Similarly, when install pyyaml first and then install aiida-core, no problem:
This leads me to conclude that that this is primarily an issue within the CI environment, but we can of course not rule out that this problem occurs in some user environments as well. |
@sphuber Based on my tests I would like to proceed with option 2, any objections? |
Fine for me to move ahead with option 2 |
As a stop-gap solution for issue aiidateam#4256.
Since builds are no longer failing, do we consider this issue closed? |
I kept it open when I merged the PR because you didn't mark it as fixing this issue. I thought this was intentional because you called it a "stepping stone" to actually solving the issue. Which is probably by relaxing the |
I was unsure myself whether we should consider this closed. I've added the |
We already reported in a couple of places that installing AiiDA in the latest Github action runners with python 3.7.8 fails actions/runner-images#1202
The reason is that, for some reason, the downgrade from pyyaml 5.3.2 to pyyaml 5.1.2 fails in that environment.
While we've temporarily switched to the python 3.8 environments for the CI tests that run on every commit, the nightly test install runs are still failing.
There are a couple of options for working around this issue:
setup-python
actionv2
and fix the version of the python environment to 3.7.7Since 1) is a bit of a hack, and this issue isn't really our fault, I would suggest 2).
3) might still take a bit of time since this update apparently first has to happen in plympy.
@csadorf @sphuber @chrisjsewell Sounds good?
The text was updated successfully, but these errors were encountered: