⚡ Fix test discovery and reduce build overhead - #2459
Merged
Conversation
Restore runtime-dependent QDMI parameterized tests and replace workload-based lifecycle timing with controlled worker synchronization. Retain simulation and verifier coverage while reducing allocation, process, compiler, and wheel duplication costs. Pin the QDMI timestamp fix and shared workflow changes for cached stubs and isolated Stable ABI wheel tests. Record local validation and the remaining hosted CI boundary. Assisted-by: GPT-6 via Codex
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Remove the wheel matrix integration and restore standard Nox source installations. Retain pytest parallelism and stub sccache. Update the QDMI pin to the CMake policy regression fix. Assisted-by: GPT-6 via Codex
Track the minimal QDMI header fix and the merged stub-cache workflow. Align the audit with the removed QDMI generation test. Assisted-by: GPT-6 via Codex
Assisted-by: GPT-6 via Codex
Submit a QIR barrier through the public device API and restore shared-library test linkage. Remove production test friendship and object-target scaffolding, and drop additional CD file triggers. Assisted-by: GPT-6 via Codex
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 AI text below 🤖
Description
QDMI test discovery could report green while omitting 176 parameterized client cases. Defer discovery until runtime manifests have been copied, and address the build and test costs measured in the profiling audit.
QDMI #537 and Workflows #462 are merged; both dependencies are pinned to their merge commits. Installed QDMI 1.3.3 remains supported, while the fetched dependency uses the header-generation fix.
Local validation: full Release CTest suite (3,142 entries, one intentional skip, plus all individual cases in the two grouped XML reports); 63 DDSIM tests against the shared device in both Debug and Release, plus five shuffled Debug repetitions of all seven lifecycle/status tests; wheel compatibility across Python 3.11–3.14 with current/minimum dependencies and version-dependent skips; normal source-building Nox tests/minimums on Python 3.14; wheel-content checks without the duplicate exemption; 247 QDMI tests and a CMake consumer from a relocated wheel; required lint and whole-changed-file C++ lint. Reconfiguration preserves all eight generated headers and Ninja reports no work.
The local wheel shrank from about 119 MiB to 55 MiB. The controlled sampling experiment reduced serial Python test time from 28.9 to 10.6 seconds. These are local measurements, not hosted CI speedup claims. Hosted platform wheels and cache behavior are pending. The audit records the measurement boundaries and pre-existing warnings from a supplementary header scan.
GPT-6 via Codex implemented and validated this change at the maintainer's request. No standalone changelog entry was added for unreleased v4 functionality, following repository policy.
Checklist
If PR contains AI-assisted content:
🤖 *AI text below* 🤖(titles are exempt).