Skip to content

Fixed Python Bindings for the CUDA Stream Protocol#202

Merged
msimberg merged 2 commits intoghex-org:masterfrom
philip-paul-mueller:fixed_protocol
Mar 26, 2026
Merged

Fixed Python Bindings for the CUDA Stream Protocol#202
msimberg merged 2 commits intoghex-org:masterfrom
philip-paul-mueller:fixed_protocol

Conversation

@philip-paul-mueller
Copy link
Copy Markdown
Collaborator

This error was found in PR#198.

@philip-paul-mueller
Copy link
Copy Markdown
Collaborator Author

Need to generate a new GHEX version.

@philip-paul-mueller
Copy link
Copy Markdown
Collaborator Author

Once this PR is merged ICON4Py must be updated, see PR.

@msimberg msimberg merged commit 848c604 into ghex-org:master Mar 26, 2026
11 of 12 checks passed
philip-paul-mueller added a commit to C2SM/icon4py that referenced this pull request Mar 27, 2026
In [PR#980](#980) introduced streams
into the halo exchanges. For this also `DEFAULT_STREAM`, which models the
default stream and implements the [CUDA Stream Protocol](https://nvidia.github.io/cuda-python/cuda-core/latest/interoperability.html#cuda-stream-protocol). However, the original
implementation identified as protocol version `1` instead of version `0`.
Because of a related bug in [GHEX](ghex-org/GHEX#202)
this error was hidden.

This PR fixes the Python implementation and also updates GHEX.
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.

2 participants