Repository navigation
Fix text drawer bindings for control-flow measurements - #17374
Phil Pinto (philpinto) wants to merge 2 commits into
Conversation
Resolve nested block classical bits to their enclosing circuit bindings when drawing measurements. Add regression coverage and a release note for Qiskit issue Qiskit#15822. Code and text developed with OpenAI Codex assistance.
qiskit-bot
added
the
Community PR
|
|
|
Thank you for opening a new pull request. Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone. One or more of the following people are relevant to this code:
|
|
Luciano Bello (@1ucian0) I noticed this was moved to “Waiting for contributor,” but I couldn’t find a review comment explaining what’s needed. I’ve updated the branch with main. Is there anything else you’d like me to address? |
Summary
Fixes #15822.
Resolve control-flow block classical bits to their outer circuit bindings when drawing measurements. Preserve bindings through nested blocks and restore the enclosing binding map after each block. Add a release note and regression coverage comparing composed circuits with equivalent builder-created circuits across eight combinations of nesting, register usage, and bundling, including permuted destination bits.
Validation
Related work
This addresses the existing issue; an earlier proposal, #16060, was closed without merging. This patch includes nested binding resolution and exact-diagram regression checks.
AI/LLM disclosure