-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Fix display of Box End after transpile in text drawer #14278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
One or more of the following people are relevant to this code:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This lgtm, thanks for fixing this
Pull Request Test Coverage Report for Build 14799279952Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Ed - I just hit this exact same bug with other control-flow constructs while writing a new version of SabreSwap
too, and it was making it hard to debug!
Always happy to help - for the easy ones... |
* Fix typing-extensions * Fix text drawer flow end display * Lint * Add reno --------- Co-authored-by: Jake Lishman <[email protected]> (cherry picked from commit 3e96ade)
* Fix typing-extensions * Fix text drawer flow end display * Lint * Add reno --------- Co-authored-by: Jake Lishman <[email protected]> (cherry picked from commit 3e96ade) Co-authored-by: Edwin Navarro <[email protected]>
@Mergifyio backport stable/1.4 |
✅ Backports have been created
|
* Fix typing-extensions * Fix text drawer flow end display * Lint * Add reno --------- Co-authored-by: Jake Lishman <[email protected]> (cherry picked from commit 3e96ade) # Conflicts: # test/python/visualization/test_circuit_text_drawer.py
Summary
Fix #14271
Details and comments
This fixes a problem when the text drawer incorrectly displayed the End of a Box after a transpile.
Now produces