-
Notifications
You must be signed in to change notification settings - Fork 190
Update cuda.bindings
to 13.0.0
#792
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Rebase to CTK 13.0 RC14
…ersion_13 Make previously overlooked 12.9.0 → 13.0.0 changes
* Update SUPPORTED_WINDOWS_DLLS: kitpicks/cuda-r13-0/13.0.0/013/local_installers/cuda_13.0.0_windows.exe * Update SUPPORTED_LINUX_SONAMES: kitpicks/cuda-r13-0/13.0.0/013/local_installers/cuda_13.0.0_580.31_linux.run * 013 → 014: SUPPORTED_LINUX_SONAMES unchanged * 013 → 014: SUPPORTED_WINDOWS_DLLS unchanged * cybind update with 13.0.0 headers (014) * Bump cuda/bindings/_version.py → 13.0.0 * test_nvjitlink.py: remove sm_60, add sm_100 * Updates from cybind after removing all 11.x headers (affects "automatically generated" comments only). * Add new toolshed/reformat_cuda_enums_as_py.py (reads cuda.h, driver_types.h headers directly). * Use new toolshed/reformat_cuda_enums_as_py.py to regenerate driver_cu_result_explanations.py, runtime_cuda_error_explanations.py * Use `driver.cuDeviceGetUuid()` instead of `driver.cuDeviceGetUuid_v2()` with CTK 13. * Adjustments for locating nvvm directory in CTK 13 installations.
* Add missing error handling (tests/test_nvjitlink.py) * Add missing `const` in cudaMemcpyBatchAsync call (cuda/bindings/runtime.pyx.in) * Add qa/13.0.0/01_linux.sh * Remove qa/13.0.0/01_linux.sh after it was moved to a new upstream qa branch. * Strictly correct casts for cudaMemcpyBatchAsync (generated by cython_gen). * Pragmatic minimal fix for cudaMemcpyBatchAsync casts (works with Linux and Windows). (generated with cython-gen)
…am bindings" This reverts commit 104abbd.
Fix accident from updating `SUPPORTED_WINDOWS_DLLS` for CTK 13
Cuda 13 soname updates
) * Linux update from cuda_13.0.0_580.46_kitpicks025_linux.run: no-op b/o NVIDIA/cuda-python-private#95 * Windows update from cuda_13.0.0_kitpicks025_windows.exe
…s overlooked. Direct commit for simplicity.
…VIDIA#94) * CCCL_INCLUDE_PATH fixes in test_event.py, test_launcher.py * Add new file (accidentally missing in a prior commit). * Fix pre-commit errors in new tests/helpers.py * 12→13 compatibility fixes in cuda/core/experimental/_graph.py * CTK 12 compatibility (tests/test_cuda_utils.py) * Make the cuda/core/experimental/_graph.py changes backwards compatible. * Do not try to hide `13` in cuda_core/tests/test_cuda_utils.py * More elegant handling of `CCCL_INCLUDE_PATHS` in cuda_core/tests/helpers.py * Remove stray empty line (cuda_core/tests/conftest.py). * Fix logic error computing CCCL_INCLUDE_PATHS in cuda_core/tests/helpers.py
/ok to test |
@rwgk, there was an error processing your request: See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/ |
/ok to test de1e83a |
There is still a problem with I'm puzzled, because it works interactively (I tried both Linux and Windows). I'll continue working on this in about one hour; good chance that I can keep working on it then until I wrestle this down for good. |
/ok to test e18a5e8 |
I started the testing to see where we stand with the CI (mainly with Windows). I know from local testing that we're still up against this error (Linux):
I need to work on understanding why this happens, because this looks as expected:
But why then the error? |
Windows passes! But for Linux, I think there is a bug/oversight in the For comparison, with CTK 12:
Now with CTK 13:
I'm afraid we have to work around that somehow. |
To resolve this issue: NVIDIA#792 (comment)
/ok to test ff339b6 |
For completeness, this ChatGPT conversation explains how I arrived at commit ff339b6: https://chatgpt.com/share/6892fd31-aafc-8008-a461-58ff0c602a0a |
/ok to test 37ef8e0 |
Since the CI was already green and the latest changes were doc-only, let me admin-merge this and run the CI in #795. |
|
Description
Close #791.
Checklist