-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade to python 3.12 #2292
Upgrade to python 3.12 #2292
Conversation
012200b
to
c5d44b5
Compare
In the Github Actions we get the following error
However when looking in
Where we can see the file |
04250fa
to
c5d44b5
Compare
Does it make any difference if you remove |
I tried this locally in |
So in version |
Ive set the env variable at the stage just before building the dev environment and it seems to do the job! |
ddfb8aa
to
d6e2bcb
Compare
Looks good from some quick local testing, but lets wait until the pyinstaller thing is understood. |
902e0a8
to
f3f8fbc
Compare
PyInstaller now builds without CUDA errors, worked out that some files were being missed in |
f3f8fbc
to
3b21da6
Compare
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 good. I've run system and unit tests locally. Run operations and reconstruction loclaly. Built a pyinstaller and tested it.
Issue
Closes #2267
Requires #2280 and #2291
Description
This PR upgrades MI to be built with
Python 3.12
, the latest version of Python.This PR will need to be rebased after #2280 is merged!
Testing
make check
make test-system
Acceptance Criteria
Run all tests and check that MI runs as normal without any issues or warnings, this transition should be seemless.
Documentation
Release note