Skip to content

Generator changes#2348

Merged
rwgk merged 2 commits into
NVIDIA:mainfrom
mdboom:cybind-preamble
Jul 13, 2026
Merged

Generator changes#2348
rwgk merged 2 commits into
NVIDIA:mainfrom
mdboom:cybind-preamble

Conversation

@mdboom

@mdboom mdboom commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

This brings in a few recent changes in the generator:

  • Generating platform-specific files, rather than using templates
  • Dropping the gil for all API calls (now for functions returning char *)
  • Not including version information in fast_enum.py

@mdboom mdboom requested a review from rwgk July 13, 2026 18:35
@github-actions github-actions Bot added the cuda.bindings Everything related to the cuda.bindings module label Jul 13, 2026
@mdboom mdboom mentioned this pull request Jul 13, 2026
@mdboom mdboom added the P0 High priority - Must do! label Jul 13, 2026
@mdboom mdboom self-assigned this Jul 13, 2026
@mdboom mdboom added this to the cuda.bindings next milestone Jul 13, 2026
@github-actions

This comment has been minimized.

@rwgk rwgk 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.

I'll go ahead and merge this.

It'd be great if you could add the MR numbers to the bullet points in the PR description. That'd make it much easier for us to make the connections back in the future, and will also make it easier to review update PRs like this one.

@rwgk rwgk merged commit ffbdc7b into NVIDIA:main Jul 13, 2026
108 of 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.

leofang pushed a commit that referenced this pull request Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindings Everything related to the cuda.bindings module P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants