Skip to content

build: add python 3.13 to test matrix #3

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

Merged
merged 3 commits into from
Mar 15, 2025
Merged

Conversation

ships
Copy link
Contributor

@ships ships commented Mar 10, 2025

Though this package is specified as Python 3.9 or greater, we are interested in expanding to use python 3.13, so this PR adds that to the test matrix for compatibility confidence.

Because all libraries we know of that use this package have separate dependencies on 3.12+ , this package's tested support for 3.9 - 3.11 is OPTIONAL and may be dropped.

@sritchie , we over here expect to be responsible for these PRs approvals and merging, but tagging you as a reviewer in case you want to adjust course as we get that flow off the ground.

Copy link

gitstream-cm bot commented Mar 10, 2025

🥷 Code experts: no user matched threshold 10

Joaoloula has most 🧠 knowledge in the files.

See details

.github/workflows/python-package.yml

Knowledge based on git-blame:

poetry.lock

Knowledge based on git-blame:

pyproject.toml

Knowledge based on git-blame:
Joaoloula: 58%

To learn more about /:\ gitStream - Visit our Docs

@ships ships force-pushed the ships/add-python-versions branch from a01c31b to 5a34b15 Compare March 11, 2025 19:31
@ships ships force-pushed the ships/add-python-versions branch from 5a34b15 to 9cae010 Compare March 11, 2025 19:35
@ships ships requested review from paxan and sritchie March 11, 2025 19:42
Copy link
Contributor

@paxan paxan left a comment

Choose a reason for hiding this comment

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

:shipit:

@@ -29,7 +28,7 @@ requires-python = ">= 3.9"
[project.optional-dependencies]
cuda = [
"jax[cuda12]>=0.4.29",
"jaxlib[cuda12_pip]>=0.4.29",
"jaxlib[cuda12]>=0.4.29",
Copy link
Contributor

Choose a reason for hiding this comment

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

Ha! Thank you for fixing this. I've been seeing warnings about non-existence of [cuda12_pip] during some local builds on my machine.

@sritchie sritchie merged commit 55d956f into main Mar 15, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants