Skip to content

MAINT: Upgrade stubgen-pyx to 0.2.19 - #2653

Merged
mdboom merged 4 commits into
NVIDIA:mainfrom
mdboom:stubgen-0.2.19-update
Aug 19, 2026
Merged

MAINT: Upgrade stubgen-pyx to 0.2.19#2653
mdboom merged 4 commits into
NVIDIA:mainfrom
mdboom:stubgen-0.2.19-update

Conversation

@mdboom

@mdboom mdboom commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

This is basically just an upgrade of stubgen-pyx to 0.2.19.

Notably, the Windows-specific bugs that toolshed/run_stubgen_pyx.py was working around are now fixed upstream, so that helper script is entirely removed.

A few minor changes were made to workaround shortcomings in the new release, and one mypy check over _dlpack.pyx is suppressed to workaround this issue: jon-edward/stubgen-pyx#60. We can come back to that and stop suppressing the issue after a future release of stubgen-pyx.

@mdboom mdboom self-assigned this Aug 17, 2026
@mdboom mdboom added CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module labels Aug 17, 2026
@github-actions

This comment has been minimized.

@mdboom mdboom added this to the cuda.core next milestone Aug 17, 2026
Temporary diagnostic to see what content actually differs when the
stubgen-pyx-cuda-core hook reports modified files on Windows CI.
@mdboom
mdboom force-pushed the stubgen-0.2.19-update branch 2 times, most recently from 14fecc1 to 5a6538b Compare August 18, 2026 16:53
@mdboom
mdboom force-pushed the stubgen-0.2.19-update branch from 5a6538b to b29e21d Compare August 18, 2026 16:55
@mdboom
mdboom requested review from Andy-Jost and rparolin and removed request for Andy-Jost August 18, 2026 21:23

@juenglin juenglin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@mdboom
mdboom merged commit f5fafa3 into NVIDIA:main Aug 19, 2026
109 checks passed
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

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

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

Labels

CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants