FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Move runtime's internal and cython layers to cybind by mdboom · Pull Request #2261 · NVIDIA/cuda-python · GitHub

/ cuda-python Public

Move runtime's internal and cython layers to cybind - #2261

Merged
mdboom merged 16 commits into
NVIDIA:mainfrom
mdboom:cybind-runtime
Jul 14, 2026
ConversationCommits (16)ChecksFiles changed
Merged

Move runtime's internal and cython layers to cybind#2261
mdboom merged 16 commits into
NVIDIA:mainfrom
mdboom:cybind-runtime

Conversation

mdboom commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This move the internal and cython layers of the runtime module to the new generator.

This has been vetted with the latest version of the check_cython_abi.py script in #2242. It has also been tested by taking a cuda.bindings.runtime module built on main and installing it alongside the lower layers built from this branch, and the unit tests pass.

It will probably be easiest to review this first on the generator side.

copy-pr-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

github-actions Bot added the cuda.bindings Everything related to the cuda.bindings module label Jun 25, 2026

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

1 similar comment

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

This comment has been minimized.

Sign in to view

mdboom commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test

mdboom marked this pull request as ready for review June 26, 2026 19:17
mdboom self-assigned this Jun 26, 2026
mdboom added this to the cuda.bindings next milestone Jun 26, 2026
leofang self-requested a review June 27, 2026 05:04
leofang added enhancement Any code-related improvements P0 High priority - Must do! labels Jun 27, 2026
mdboom enabled auto-merge (squash) July 14, 2026 15:51
mdboom merged commit c4a5647 into NVIDIA:main Jul 14, 2026
105 checks passed

github-actions Bot commented Jul 14, 2026

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

rparolin mentioned this pull request Aug 28, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

kkraus14 kkraus14 approved these changes
leofang Awaiting requested review from leofang

Assignees

mdboom

Labels

cuda.bindings Everything related to the cuda.bindings module enhancement Any code-related improvements P0 High priority - Must do!

Projects

None yet

Milestone

cuda.bindings 13.5.0 & 12.9.10

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL