Skip to content

Use only public .handle to access cuda-core objects#794

Merged
brandon-b-miller merged 5 commits intoNVIDIA:mainfrom
brandon-b-miller:fix-core-private-handles
Feb 23, 2026
Merged

Use only public .handle to access cuda-core objects#794
brandon-b-miller merged 5 commits intoNVIDIA:mainfrom
brandon-b-miller:fix-core-private-handles

Conversation

@brandon-b-miller
Copy link
Copy Markdown
Contributor

Fix an issue where we're erroneously accessing the _handle of cuda-core objects rather than the .handle.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Feb 18, 2026

Automatic reviews are disabled for this repository.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Feb 18, 2026

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

Contributors can view more details about this message here.

@brandon-b-miller
Copy link
Copy Markdown
Contributor Author

/ok to test

@kkraus14
Copy link
Copy Markdown
Contributor

@brandon-b-miller it looks like we only introduced it as a public attribute in the upcoming 0.6 release where we'd need to handle it conditionally it seems.

@brandon-b-miller
Copy link
Copy Markdown
Contributor Author

/ok to test

@brandon-b-miller
Copy link
Copy Markdown
Contributor Author

/ok to test

@brandon-b-miller
Copy link
Copy Markdown
Contributor Author

/ok to test

@brandon-b-miller brandon-b-miller merged commit 62edb06 into NVIDIA:main Feb 23, 2026
206 of 208 checks passed
@brandon-b-miller brandon-b-miller deleted the fix-core-private-handles branch February 23, 2026 13:32
gmarkall added a commit to gmarkall/numba-cuda that referenced this pull request Mar 2, 2026
- Revert exception hierarchy unification (NVIDIA#634) and its fix (NVIDIA#796) (NVIDIA#816)
- fix(errors): restore CUDA exception hierarchy to avoid slow string compilation (NVIDIA#796)
- Aligned NumPy dtypes caching (NVIDIA#792)
- Fix mixed C ABI / Numba ABI internal-call lowering (NVIDIA#781, NVIDIA#789) (NVIDIA#782)
- Disable legilize return type for inline always (NVIDIA#812)
- Unpin nvjitlink from ctk version (NVIDIA#809)
- Fix int64 elements DWARF encoding in UniTuple types (NVIDIA#807)
- Fix incorrect DWARF type encodings for i8 and discriminator types (NVIDIA#806)
- Find CUDA headers via pathfinder (NVIDIA#771)
- Remove most of `drvapi.py` in favor of direct `cuda-python` usage (NVIDIA#784)
- Use only public `.handle` to access `cuda-core` objects (NVIDIA#794)
- Fix np.dtype overload signature drift (NVIDIA#797)
- Remove unused cudf patch (NVIDIA#786)
- Bump `cudf` version to `26.02` in thirdparty tests (NVIDIA#785)
- Fix pnputil to only restart NVIDIA display adapters (NVIDIA#779)
- Clear `rtsys` during context reset (NVIDIA#783)
- Remove `enums` and unused ctypes code that required it (NVIDIA#775)
- Fix CI checks job to also fail on dependency failures (NVIDIA#777)
@gmarkall gmarkall mentioned this pull request Mar 2, 2026
gmarkall added a commit that referenced this pull request Mar 2, 2026
- Revert exception hierarchy unification (#634) and its fix (#796)
(#816)
- fix(errors): restore CUDA exception hierarchy to avoid slow string
compilation (#796)
- Aligned NumPy dtypes caching (#792)
- Fix mixed C ABI / Numba ABI internal-call lowering (#781, #789) (#782)
- Disable legilize return type for inline always (#812)
- Unpin nvjitlink from ctk version (#809)
- Fix int64 elements DWARF encoding in UniTuple types (#807)
- Fix incorrect DWARF type encodings for i8 and discriminator types
(#806)
- Find CUDA headers via pathfinder (#771)
- Remove most of `drvapi.py` in favor of direct `cuda-python` usage
(#784)
- Use only public `.handle` to access `cuda-core` objects (#794)
- Fix np.dtype overload signature drift (#797)
- Remove unused cudf patch (#786)
- Bump `cudf` version to `26.02` in thirdparty tests (#785)
- Fix pnputil to only restart NVIDIA display adapters (#779)
- Clear `rtsys` during context reset (#783)
- Remove `enums` and unused ctypes code that required it (#775)
- Fix CI checks job to also fail on dependency failures (#777)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants