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

Add support for Python 3.12 #312

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Conversation

jameslamb
Copy link
Member

Description

Contributes to rapidsai/build-planning#40

This PR adds support for Python 3.12.

Notes for Reviewers

This is part of ongoing work to add Python 3.12 support across RAPIDS.
It temporarily introduces a build/test matrix including Python 3.12, from rapidsai/shared-workflows#213.

A follow-up PR will revert back to pointing at the branch-24.10 branch of shared-workflows once all
RAPIDS repos have added Python 3.12 support.

This will fail until all dependencies have been updates to Python 3.12

CI here is expected to fail until all of this project's upstream dependencies support Python 3.12.

This can be merged whenever all CI jobs are passing.

@jameslamb jameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Sep 4, 2024
@github-actions github-actions bot added the Python label Sep 4, 2024
@jameslamb jameslamb changed the title Add support for Python 3.12 WIP: Add support for Python 3.12 Sep 4, 2024
@jameslamb
Copy link
Member Author

Restarted CI here now that there are Python 3.12 packages for raft: rapidsai/raft#2428

Will put this up for review once I see enough of CI passing.

@jameslamb jameslamb changed the title WIP: Add support for Python 3.12 Add support for Python 3.12 Sep 10, 2024
@jameslamb jameslamb marked this pull request as ready for review September 10, 2024 13:27
@jameslamb jameslamb requested review from a team as code owners September 10, 2024 13:27
@jameslamb jameslamb requested review from KyleFromNVIDIA and bdice and removed request for KyleFromNVIDIA September 10, 2024 13:27
Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

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

We may also want to update

"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
to drop 3.9 and add 3.12?

(edit: it was just added 13 hours ago in #275 -- you didn't miss this)

@bdice
Copy link
Contributor

bdice commented Sep 10, 2024

I am going to merge this PR and then do a follow-up for updating the new cuvs_bench changes. CI has already passed here and we don't need to delay merging any further (these changes should be tested with other PRs going forward).

@bdice
Copy link
Contributor

bdice commented Sep 10, 2024

/merge

@rapids-bot rapids-bot bot merged commit 6f0070d into rapidsai:branch-24.10 Sep 10, 2024
54 checks passed
@jameslamb
Copy link
Member Author

Ok thanks very much! Yep I hadn't noticed that new cuvs_bench stuff package, thanks for catching it.

@bdice
Copy link
Contributor

bdice commented Sep 10, 2024

Follow-up here: #318

rapids-bot bot pushed a commit that referenced this pull request Sep 10, 2024
This is a follow-up for #275 and #312. I updated `cuvs_bench`'s `pyproject.toml` to drop Python 3.9 and add Python 3.12.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #318
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants