Prepare cuda-bindings 12.9.8 release - #2814
Conversation
rwgk
left a comment
There was a problem hiding this comment.
I'm still looking with codex as well.
| Prerelease feature | ||
| ------------------ | ||
|
|
||
| A new version of the ``nvrtc`` API is available as ``cuda.bindings._v2.nvrtc``. The |
There was a problem hiding this comment.
| A new version of the ``nvrtc`` API is available as ``cuda.bindings._v2.nvrtc``. The | |
| This release introduces an experimental, alternative ``cuda.bindings._v2.nvrtc`` API. The |
The main idea is to insert the word "experimental" early.
There was a problem hiding this comment.
Correction after checking the branch and the PR-built wheel: it seems this whole section should be removed. _v2.nvrtc is a 13.4-only preview API and is not packaged by 12.9.x.
| Prerelease feature | ||
| ------------------ |
There was a problem hiding this comment.
xref: #2836
| Prerelease feature | |
| ------------------ | |
| Preview feature | |
| --------------- |
| (`PR #2581 <https://github.com/NVIDIA/cuda-python/pull/2581>`_) | ||
| * Fixed a crash in ``nvml.system_event_set_wait`` caused by calling | ||
| ``resize()`` on a non-owning ``SystemEventData_v1._data`` view. | ||
| (`PR #2690 <https://github.com/NVIDIA/cuda-python/pull/2690>`_) | ||
| * ``get_cuda_native_handle`` no longer misreports a ``KeyError`` raised from | ||
| within a registered getter as an "Unknown type" error. | ||
| (`PR #2551 <https://github.com/NVIDIA/cuda-python/pull/2551>`_) | ||
| * Fixed ``cuFile`` status checking to no longer raise ``cuFileError`` | ||
| spuriously when ``CUfileError_t.cu_err`` is set on a non-error path (for | ||
| example, BAR-size queries on GH200 systems). | ||
| (`PR #2530 <https://github.com/NVIDIA/cuda-python/pull/2530>`_) | ||
| * Made ``param_packer.feed()`` safe under free-threaded Python by moving its | ||
| internal state initialization to import time. | ||
| (`PR #2417 <https://github.com/NVIDIA/cuda-python/pull/2417>`_) | ||
|
|
There was a problem hiding this comment.
The formatting here seems off? PR XXXX comes before the bullet point
There was a problem hiding this comment.
also it doesn't seem that the style is consistent with past release notes (we don't link to PRs), although this is just a minor nit
leofang
left a comment
There was a problem hiding this comment.
Q: We probably should also update cuda_python/docs in this branch?
rwgk
left a comment
There was a problem hiding this comment.
The cibuildwheel 4.2.0 bump means the cp315/cp315t wheels are now built with CPython 3.15.0rc1, but the post-build/test environments still force 3.15.0-beta.2 in:
.github/workflows/build-wheel.yml:233.github/workflows/test-wheel-linux.yml:220.github/workflows/test-wheel-windows.yml:210
This crosses a prerelease ABI change and is now causing all six 3.15/3.15t wheel-test jobs to crash during extension import—segmentation faults on Linux and access violations on Windows. Could we update or remove these three beta.2 overrides so the build and test interpreters match?
|
|
||
| ``cuda-bindings`` 12.9.8 Release notes | ||
| ====================================== | ||
|
|
There was a problem hiding this comment.
This appears to have been based on the 13.4.1 notes rather than the changes actually shipped on 12.9.x. The #2581, #2551, and #2417 changes and the _v2.nvrtc preview API (#2398) are not present on this branch; #2690 and #2530 are present. Conversely, actual changes since v12.9.7 such as #2145 and #2382 are not mentioned. Could we rebuild these notes from the v12.9.7..12.9.x history and remove the _v2 section?
Also, I think this new page needs to be linked from cuda_bindings/docs/source/release.rst.
Regenerate files and add release notes for cuda-bindings 12.9.8