Refresh CUDA 13.4 bindings (cybind !561, !574, !580) - #2953
Conversation
|
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. |
|
/ok to test 20f1320 |
|
|
/ok to test 6f04788 |
…CHANGES) after adding commit 797a08ab9d2262fb11ba7d7207e1a66495f818fc to cybind !580
|
All of the ABI breakage in this PR is at the I can see how it's confused by As for this, this is an API change, but required for a bugfix:
We should add a breaking change note for this (I'll push another commit here). |
Description
Regenerate the CUDA 13.4 bindings using cybind MR !580.
Supersedes #2950.
All changed files are generated; none was edited by hand.
This refresh brings in three behavior changes:
88a61d6d,3ac802c488a61d6d,3ac802c4nvml.NvlinkFirmwareInfo.firmware_versionnow exposes onlynumValidEntrieselements rather than all 100 slots. Its setter accepts up to 100 elements and sets the count automatically. The generatednum_valid_entriesPython property (including its setter) is removed. — cybind !561a7b0b86aCompatibility review
The first fix adds
_ownerfields to 241 public Cythoncdef classdeclarations; regeneration also adds two fields toCUmemDecompressParams_st. These declarations change extension-type layouts. The cuda.bindings support policy exposes the Cythoncimportlayer and warns that a new minor release may require downstream rebuilds. The repository's ABI checker treats changed class sizes and fields as errors. If these bindings are released as a 13.4.x patch, please assess whether prebuilt Cython consumers can continue using them or whether this fix needs an ABI-preserving approach or a minor release.The pointer-array getters' change from live views to copies and removal of
NvlinkFirmwareInfo.num_valid_entriesalso need Python API compatibility review for a patch release. The latter is an additional effect of regeneration, not part of the two driver/runtime lifetime fixes.