Skip to content

[DO NOT MERGE] Test against cuda-quantum 0.15.x#702

Draft
anjbur wants to merge 34 commits into
NVIDIA:mainfrom
anjbur:0.15.1-build-test
Draft

[DO NOT MERGE] Test against cuda-quantum 0.15.x#702
anjbur wants to merge 34 commits into
NVIDIA:mainfrom
anjbur:0.15.1-build-test

Conversation

@anjbur

@anjbur anjbur commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Description

Draft PR enabling CI tests against cuda-quantum release 0.15.0 (plus a few specific commits). See the branch git history for details on which commits from cudaqx were reverted.

In addition to the PR CI, I also successfully built the cudaqx-dev image locally. I have not run the CI action to build the dev image.

Runtime / performance impact

Self-review checklist

Please confirm each item before requesting review. Check [x] or strike
through and explain.

Before requesting review

  • I reviewed my own full diff in GitHub or my editor.
  • PR is in Draft if it is not yet ready for review.
  • Temporary / debugging changes have been removed.
  • Local test logs reviewed; no unexplained warnings or errors.
  • CI logs reviewed; no unexplained warnings or errors.
  • Full CI has been run.

Scope and size

  • PR is under ~1000 lines, or an exception is justified in the description.
  • Refactoring-only changes are isolated in their own PR(s).
  • No existing tests were disabled or modified just to make this PR pass
    (if so, an issue has been raised).

Tests

  • New functionality has new tests.
  • Tests fail if the new functionality is broken (including crashes), not
    just when it is missing.
  • Negative tests added where exceptions are expected.
  • Truth data added where simple EXPECT_* / assert checks are
    insufficient for algorithmic correctness.
  • CI runtime impact considered; team notified if significant.

Documentation

  • Public-facing APIs have Doxygen docs.
  • User-visible behavior changes have public docs, or a follow-up is
    tracked.

Code style

  • Naming follows the existing convention (snake_case vs camelCase) for
    the area being modified.

Dependencies

  • No new third-party dependencies, or the team has been notified and
    OSRB tickets filed.

anjbur added 30 commits July 15, 2026 18:50
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit b974c3f.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit 78b8d7d.

Signed-off-by: Angela Burton <angelab@nvidia.com>
)"

This reverts commit 2cdc9df.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…-server test (NVIDIA#673)"

This reverts commit f784db3.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…B decoding-server test (NVIDIA#670)"

This reverts commit fa6c200.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit 6c688b5.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…oder configuration (NVIDIA#679)"

This reverts commit 35ee248.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…chedule (NVIDIA#672)"

This reverts commit 22772f7.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit efbb6f9.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…sliding window decoder (NVIDIA#656)"

This reverts commit 131c4d0.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit 9c5c230.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…NVIDIA#690)"

This reverts commit 2cd873b.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit 7df80cb.

Signed-off-by: Angela Burton <angelab@nvidia.com>
This reverts commit 2636cf0.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…LDPC decoding (NVIDIA#631)"

This reverts commit 22d7d3f.

Signed-off-by: Angela Burton <angelab@nvidia.com>
…ransport integration

Signed-off-by: Angela Burton <angelab@nvidia.com>
…ntegration

Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
…integration

Signed-off-by: Angela Burton <angelab@nvidia.com>
…integration

Signed-off-by: Angela Burton <angelab@nvidia.com>
…VIDIA#609)". Resolve NVIDIA#609 revert conflicts by restoring the legacy host-call and direct-decoding paths while retaining the GPU device-placement fixes from NVIDIA#690 and NVIDIA#678.

This reverts commit 927ec7f.

Signed-off-by: Angela Burton <angelab@nvidia.com>
anjbur and others added 4 commits July 15, 2026 21:28
… (927ec7f), required by CUDA-Q PR #4682 (1612f9fe).

Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Apply cuda-quantum CMake patch with reduced context

build_cudaq.sh patches CMakeLists.txt (Python find_package Development ->
Development.Module), but upstream cuda-quantum #4698 restructured the
python/nanobind block, shifting the surrounding context so `git apply` can no
longer locate the hunk (hit when .cudaq_version points at a ref that includes
Non-interactive on purpose: `patch` can hang on a File-to-patch prompt in CI.

Signed-off-by: Chuck Ketcham <cketcham@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
Signed-off-by: Angela Burton <angelab@nvidia.com>
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