Skip to content
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

Fix mpl circuit drawer with fold=-1 hanging with ControlFlow ops #12016

Merged
merged 13 commits into from
Mar 18, 2024

Conversation

enavarro51
Copy link
Contributor

Summary

Fixes #12012

Details and comments

This PR fixes a hang in the mpl circuit drawer when fold is set to -1 and there are ControlFlow ops in the circuit.

@enavarro51 enavarro51 requested review from nonhermitian and a team as code owners March 15, 2024 16:48
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

@coveralls
Copy link

Pull Request Test Coverage Report for Build 8299605997

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 5 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.001%) to 89.279%

Changes Missing Coverage Covered Lines Changed/Added Lines %
qiskit/visualization/circuit/matplotlib.py 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/expr.rs 1 93.81%
crates/qasm2/src/lex.rs 4 91.69%
Totals Coverage Status
Change from base Build 8297051089: -0.001%
Covered Lines: 59707
Relevant Lines: 66877

💛 - Coveralls

@1ucian0 1ucian0 added Changelog: Bugfix Include in the "Fixed" section of the changelog stable backport potential The bug might be minimal and/or import enough to be port to stable labels Mar 18, 2024
Copy link
Member

@1ucian0 1ucian0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix Edwin!

@1ucian0 1ucian0 added this pull request to the merge queue Mar 18, 2024
@1ucian0
Copy link
Member

1ucian0 commented Mar 18, 2024

@Mergifyio backport stable/0.46

Copy link
Contributor

mergify bot commented Mar 18, 2024

backport stable/0.46

✅ Backports have been created

Merged via the queue into Qiskit:main with commit 43381ae Mar 18, 2024
12 checks passed
mergify bot pushed a commit that referenced this pull request Mar 18, 2024
)

* Fix typing-extensions

* Fix control flow with fold minus one

(cherry picked from commit 43381ae)

# Conflicts:
#	test/visual/mpl/circuit/test_circuit_matplotlib_drawer.py
mergify bot pushed a commit that referenced this pull request Mar 18, 2024
)

* Fix typing-extensions

* Fix control flow with fold minus one

(cherry picked from commit 43381ae)
github-merge-queue bot pushed a commit that referenced this pull request Mar 19, 2024
) (#12037)

* Fix typing-extensions

* Fix control flow with fold minus one

(cherry picked from commit 43381ae)

Co-authored-by: Edwin Navarro <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Mar 22, 2024
…kport #12016) (#12036)

* Fix mpl circuit drawer with fold=-1 hanging with ControlFlow ops (#12016)

* Fix typing-extensions

* Fix control flow with fold minus one

(cherry picked from commit 43381ae)

# Conflicts:
#	test/visual/mpl/circuit/test_circuit_matplotlib_drawer.py

* Update test_circuit_matplotlib_drawer.py

* Update test_circuit_matplotlib_drawer.py

* Update test_circuit_matplotlib_drawer.py

* Update test_circuit_matplotlib_drawer.py

* Update test/visual/mpl/circuit/test_circuit_matplotlib_drawer.py

---------

Co-authored-by: Edwin Navarro <[email protected]>
Co-authored-by: Luciano Bello <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the "Fixed" section of the changelog stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MPL drawer hangs on dynamic circuits when fold=-1
4 participants