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 a bug is StabilizerState repr #11368

Merged
merged 3 commits into from
Dec 14, 2023
Merged

Conversation

ShellyGarion
Copy link
Member

Summary

Currently, printing StabilizerState(qc) raises the following error, since the Clifford class API has been changed.
AttributeError: 'Clifford' object has no attribute 'stabilizer'

This PR fixes this bug, and adds a test to check that the printing does not raise an error.

Details and comments

@ShellyGarion ShellyGarion added the Changelog: Bugfix Include in the "Fixed" section of the changelog label Dec 4, 2023
@ShellyGarion ShellyGarion added this to the 0.45.2 milestone Dec 4, 2023
@ShellyGarion ShellyGarion requested review from ikkoham and a team as code owners December 4, 2023 10:42
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core
  • @ikkoham

@coveralls
Copy link

coveralls commented Dec 4, 2023

Pull Request Test Coverage Report for Build 7096995386

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 6 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.03%) to 87.449%

Files with Coverage Reduction New Missed Lines %
qiskit/transpiler/passes/synthesis/unitary_synthesis.py 1 90.74%
crates/qasm2/src/lex.rs 5 91.67%
Totals Coverage Status
Change from base Build 7078267006: 0.03%
Covered Lines: 59926
Relevant Lines: 68527

💛 - Coveralls

Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

Thanks Shelly. Can you add a one-line fix release note?

@mtreinish mtreinish added the stable backport potential The bug might be minimal and/or import enough to be port to stable label Dec 4, 2023
Copy link
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

Thanks!

@jakelishman jakelishman added this pull request to the merge queue Dec 14, 2023
Merged via the queue into Qiskit:main with commit 1b69780 Dec 14, 2023
13 checks passed
mergify bot pushed a commit that referenced this pull request Dec 14, 2023
* fix a bug in StabilizerState repr

* add a test to check repr does not throw an error

* add release notes

(cherry picked from commit 1b69780)
github-merge-queue bot pushed a commit that referenced this pull request Dec 14, 2023
* fix a bug in StabilizerState repr

* add a test to check repr does not throw an error

* add release notes

(cherry picked from commit 1b69780)

Co-authored-by: Shelly Garion <[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.

5 participants