Skip to content

Bump cython dependency to 2.3.5 - #2469

Merged
seberg merged 1 commit into
NVIDIA:mainfrom
seberg:cython325
Aug 2, 2026
Merged

Bump cython dependency to 2.3.5#2469
seberg merged 1 commit into
NVIDIA:mainfrom
seberg:cython325

Conversation

@seberg

Copy link
Copy Markdown
Contributor

As Keith noted, this is needed for using the py_safe_call_once definitions, Cython 3.2.5 changelog:
https://cython.readthedocs.io/en/latest/src/changes.html

(I guess the bump in the pre-commit is likely not strictly needed, but there also were no stub changes.)

This did change the docs entries to pin the same range. Not really needed, but it did seem fine to keep consistent (and my agent said it keeps the pixi locks simpler).

@github-actionsgithub-actionsBot added cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module labels Jul 31, 2026
@sebergseberg changed the title Bumpy cython dependency to 2.3.5Bump cython dependency to 2.3.5Jul 31, 2026
@sebergseberg added the P1 Medium priority - Should do label Jul 31, 2026
@sebergseberg self-assigned this Jul 31, 2026
As Keith noted, this is needed for using the `py_safe_call_once`
definitions, Cython 3.2.5 changelog:
https://cython.readthedocs.io/en/latest/src/changes.html
(I guess the bump in the pre-commit is likely not strictly needed, but
there also were no stub changes.)
@sebergseberg added this to the cuda.core 1.2.0 milestone Jul 31, 2026
@sebergseberg added the enhancement Any code-related improvements label Jul 31, 2026
@github-actions

This comment has been minimized.

@rparolinrparolin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@seberg
seberg merged commit 17e2749 into NVIDIA:mainAug 2, 2026
109 of 111 checks passed
@seberg
seberg deleted the cython325 branch August 2, 2026 15:43
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindingsEverything related to the cuda.bindings modulecuda.coreEverything related to the cuda.core modulecuda.pathfinderEverything related to the cuda.pathfinder moduleenhancementAny code-related improvementsP1Medium priority - Should do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@seberg@rparolin@juenglin