-
Notifications
You must be signed in to change notification settings - Fork 4
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
#599: Add support for python 3.12 continuation #640
Merged
tim-vd-aardweg
merged 30 commits into
main
from
feat/599-add-support-for-python-312-continueation
Jun 7, 2024
Merged
#599: Add support for python 3.12 continuation #640
tim-vd-aardweg
merged 30 commits into
main
from
feat/599-add-support-for-python-312-continueation
Jun 7, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 tasks
|
When this PR is merged and closed, #606 can also be closed as resolved by this PR. |
tim-vd-aardweg
approved these changes
May 7, 2024
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.
Review OK!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue information
It appears that numpy dropped support for python 3.8 in numpy 1.25.
In numpy 1.26 support for python 3.12 was added.
https://numpy.org/news/
The pyproject.toml file had the numpy version set to "^1.21".
I expect that it would search from this version upwards for a compatible version for the python version in use.
E.g. python 3.8 would have with a poetry install numpy 1.24, python 3.12 would have with a poetry install numpy 1.26.
However in practice this did not seem the case, when python 3.12 was used it could not find and use numpy 1.26.
Currenlty I update the pyproject.toml to use "^1.21" for python version 3.8 and above and use "^1.26" for python version 3.12 and above.
Issue TODOs
Test notes
When using poetry install in a clean python 3.12 env with
numpy="^1.21"
and the lock file removed, the following error is encountered:One thing to note is that numpy version 1.24.4 is being installed.
This version is available on pypi: https://pypi.org/project/numpy/#history
But when the release documentation of Numpy is checked, none is available for 1.24.4:
https://numpy.org/doc/stable/release/1.24.3-notes.html
https://numpy.org/doc/stable/release/1.24.4-notes.html
It might be that something is wrong with this version, causing the install to break.
Update: I did find release notes for numpy 1.24.4, but they are on the devdocs page, meaning that it appears this version was release, but not as a stable version. This might be related to the issue.
https://numpy.org/devdocs/release/1.24.4-notes.html
When using poetry install in a clean python 3.12 env with
numpy="1.*"
and the lock file removed, the following error is encountered:When using poetry install in a clean python 3.12 env with
and the lock file removed, no errors are encountered.
Black
The black version should be updated once it is available for python 3.12, currently it does not seem to be supported.
see: --target-version
https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html
Python 3.8 support
Python 3.8 will be end of life on 2024-10, do we still want to support this python version up to and after this point?
https://devguide.python.org/versions/