-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feanil/test on 3.11 #34374
feanil/test on 3.11 #34374
Conversation
37b5fb9
to
ab4bd00
Compare
96d8d45
to
ec86f91
Compare
d915d8d
to
f433751
Compare
f433751
to
8a78701
Compare
8a78701
to
b512f37
Compare
d17340e
to
288cf40
Compare
Scipy was constrained to prevent a numpy upgrade however the numpy upgrade happened a while ago and this never got updated. Now the scipy version is preventing us from going to an even newer numpy version that would be compatible with both Python 3.8 and Python 3.12 BREAKING CHANGE: This will update scipy in the codejail sandbox from 1.7.3 to a newer version. This contains many backwards incompatible changes that are all documented in the SciPy changelogs: https://docs.scipy.org/doc/scipy/release/1.8.0-notes.html
Ran `package=scipy make upgrade-package` to upgrade just this one dependency.
BREAKING CHANGE: Because numpy is available in the codejail sandbox, the update from 1.22.4 to 1.24.4 does have completed deprecations, and more details can be found in the Numpy Release Notes: https://numpy.org/doc/stable/release/1.23.0-notes.html#expired-deprecations
We do this so that we can build requirements files that could work to `pip install` on both the old (3.8) and new (3.11) versions of python.
288cf40
to
61a603c
Compare
* Update so tox will allow running on the new version. * Update so the hosted runner actually respects the python version in the matrix
61a603c
to
325a7b9
Compare
@@ -12,7 +12,7 @@ jobs: | |||
|
|||
strategy: | |||
matrix: | |||
python-version: [ '3.8', '3.12' ] | |||
python-version: [ '3.8', '3.11', '3.12' ] |
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 3.12 purposeful?
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.
When the changes were coming into edx-platform we still thought we were going directly to 3.12 so that's why it was there in the first place. I left 3.12 in since otherwise I'll just have to add it in a future PR and this one is small enough that I'm not worried.
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.
👍
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
This PR adds testing for python 3.11 wherever possible alongside python 3.8.
Major updates include the unpinning and upgrade fo SciPy and Numpy in both the platform requirements as well as the codejail (edx-sandbox) requirements. If you are using the latest edx-sandbox/base.txt to operate codejail environments, this may break your instructor code that utilizes sci-py/numpy.