Skip to content
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

Merged
merged 7 commits into from
May 2, 2024
Merged

feanil/test on 3.11 #34374

merged 7 commits into from
May 2, 2024

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Mar 14, 2024

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.

@feanil feanil force-pushed the feanil/test_on_3.11 branch 2 times, most recently from 37b5fb9 to ab4bd00 Compare March 14, 2024 18:35
@feanil feanil marked this pull request as ready for review March 14, 2024 19:24
@feanil feanil force-pushed the feanil/test_on_3.11 branch 2 times, most recently from 96d8d45 to ec86f91 Compare March 15, 2024 19:44
@feanil feanil force-pushed the feanil/test_on_3.11 branch 2 times, most recently from d915d8d to f433751 Compare April 3, 2024 13:54
@feanil feanil requested a review from kdmccormick April 4, 2024 13:15
@feanil feanil linked an issue Apr 5, 2024 that may be closed by this pull request
4 tasks
@feanil feanil force-pushed the feanil/test_on_3.11 branch 4 times, most recently from d17340e to 288cf40 Compare April 30, 2024 17:26
feanil added 5 commits May 1, 2024 10:03
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.
feanil added 2 commits May 1, 2024 10:23
* Update so tox will allow running on the new version.
* Update so the hosted runner actually respects the python version in
  the matrix
@@ -12,7 +12,7 @@ jobs:

strategy:
matrix:
python-version: [ '3.8', '3.12' ]
python-version: [ '3.8', '3.11', '3.12' ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is 3.12 purposeful?

Copy link
Contributor Author

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.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@feanil feanil merged commit c19aca1 into master May 2, 2024
218 checks passed
@feanil feanil deleted the feanil/test_on_3.11 branch May 2, 2024 12:50
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[edx-platform] Test Python 3.11
3 participants