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

update comment in a test file #11317

Closed
wants to merge 5 commits into from

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Nov 24, 2023

Not a fix, but I ran into this problem and I noticed that there was a note in one of the tests. However, the link in the notes is broken. Is #7346 the write issue that is tracking the problem?

@1ucian0 1ucian0 requested a review from a team as a code owner November 24, 2023 18:22
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core

@coveralls
Copy link

coveralls commented Nov 24, 2023

Pull Request Test Coverage Report for Build 8749665399

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 10 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.009%) to 89.274%

Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/lex.rs 4 92.37%
crates/qasm2/src/parse.rs 6 97.61%
Totals Coverage Status
Change from base Build 8748077179: 0.009%
Covered Lines: 60320
Relevant Lines: 67567

💛 - Coveralls

@1ucian0 1ucian0 added the Changelog: None Do not include in changelog label Nov 25, 2023
qc = QuantumCircuit(2)
qc.cz(0, 1)

bt_pass = BasisTranslator(std_eqlib, target_basis=None, target=self.target)
output = bt_pass(qc)
# We need a second run of BasisTranslator to correct gates outside of
# the target basis. This is a known isssue, see:
# https://qiskit.org/documentation/release_notes.html#release-notes-0-19-0-known-issues
Copy link
Member

Choose a reason for hiding this comment

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

Fwiw the updated release note link is: https://qiskit.org/documentation/legacy_release_notes.html#release-notes-0-19-0-known-issues but I know that won't be valid itself after too long. But the example there is the minimum reproducer of this problem. In practice it only comes up for manually constructed targets though, it's not a real issue as none of the backends I'm aware of have heterogeneous 1q gates.

Copy link
Member Author

@1ucian0 1ucian0 Nov 29, 2023

Choose a reason for hiding this comment

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

Thanks for recovering the original known issue! I moved it to an issue so we keep it with the rest of them and updated the comment.

@1ucian0 1ucian0 changed the title BasisTranslator needs to be run twice update comment in a test file Nov 29, 2023
auto-merge was automatically disabled June 12, 2024 09:53

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants