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

Update python to current supported versions. Also add experimental support for next python release #5503

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Fale
Copy link

@Fale Fale commented Nov 13, 2024

Description

Bump python version in the CI to current versions.

.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@Fale
Copy link
Author

Fale commented Nov 15, 2024

I've added back 3.9 and 3.10. Also I've pushed 3.12 to be ok to fail, since now it is failing. The best scenario would be to have up to 3.13 working properly, considering that 3.13 is already in Fedora 41 (released last month) and very soon other distros will move to it

@snejus
Copy link
Member

snejus commented Nov 15, 2024

I see that you're dropping support for Python 3.8 - note that this needs a bit more effort:

  1. Looking through the release notes, checking the codebase against any removals/deprecations and updating the codebase accordingly.
  2. Typing updates: we should use pyupgrade to align types used in the codebase with what Python 3.9 supports.
  3. Update of Python constraints in the package.
  4. Relevant updates in the documentation.
  5. And finally the changelog note.

That's a couple of things just off the top of my head - there's likely a couple of others.

Are you happy to go ahead with the above or would you like to limit the scope of this PR to simply add support for the most recent version without removing Python 3.8?

@Fale
Copy link
Author

Fale commented Nov 16, 2024

I've added back 3.8.

The reason why I removed it in the first place was that it is EOL, but I don't mind it being there. The part I'm interested into is the support to the newest versions. I don't have anything against the old versions :).

@snejus
Copy link
Member

snejus commented Nov 16, 2024

Deps installation on Python 3.12+ seems to be failing due to numpy 1.24.4 which does not support Python 3.12. It added support in 1.26.0, however in the same release it also removed support for Python 3.8.

Poetry resolved this numpy version because we are still supporting Python 3.8. Just tried setting minimum python version to 3.9 in the package and resolved the dependencies - numpy indeed gets upgraded, which should resolve this issue.

I will shortly submit a PR that removes Python 3.8 support and fixes this issue for you.

@snejus snejus mentioned this pull request Nov 17, 2024
3 tasks
@Fale
Copy link
Author

Fale commented Nov 17, 2024

Thanks a lot :)

As soon as the other PR is merged, I'll rebase this one :)

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.

2 participants