Skip to content

cuda.compute: Fix struct comparison (ordering matters)#7451

Merged
shwina merged 1 commit intoNVIDIA:mainfrom
shwina:fix-struct-comparison-ordering
Feb 2, 2026
Merged

cuda.compute: Fix struct comparison (ordering matters)#7451
shwina merged 1 commit intoNVIDIA:mainfrom
shwina:fix-struct-comparison-ordering

Conversation

@shwina
Copy link
Contributor

@shwina shwina commented Feb 2, 2026

Description

Fixes a bug that leads to examples failing in CI (depending on whether two particular tests are run on the same worker).

For posterity, a reliable reproducer of the issue was:

pytest tests/test_examples.py -v -k "nested_struct_tuple_construction or running_average"

Since these two tests create structs with the same members (but different order), and the first struct would be incorrectly retrieved from a cache during the execution of the second test.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@shwina shwina requested a review from a team as a code owner February 2, 2026 16:28
@shwina shwina requested a review from rwgk February 2, 2026 16:28
@github-project-automation github-project-automation bot moved this to Todo in CCCL Feb 2, 2026
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Feb 2, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

🥳 CI Workflow Results

🟩 Finished in 1h 19m: Pass: 100%/56 | Total: 18h 23m | Max: 55m 21s

See results here.

@shwina shwina merged commit 663f5cf into NVIDIA:main Feb 2, 2026
80 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Feb 2, 2026
fbusato pushed a commit to fbusato/cccl that referenced this pull request Feb 19, 2026
Co-authored-by: Ashwin Srinath <shwina@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants