-
Notifications
You must be signed in to change notification settings - Fork 85
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
Remove "warning suppression" code #1307
Conversation
Thanks for contributing to Qiskit documentation! 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. Thanks! 🙌 |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
the tutorial "Submitting transpiled circuits" needs to be updated to V2. |
@Eric-Arellano and @frankharkins, this should be ready for review, but one of the cells in VQE is timing out in CI. Can someone help me address that so it can be completed? It runs fine on my local simulator, so it shouldn't be timing out. |
Running notebooks with the `FakeKyoto` backend was occassionally taking a very long time (see #1307 (comment)). We believe this is due to some notebooks taking advantage of the large number of qubits it supports. This PR switches to Qiskit's `GenericBackendV2`. This should be much faster and gives us more fine-grained control over the test backend properties.
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.
Thank you!
@beckykd @frankharkins Thanks for fixing #1331. The callback section can be removed entirely since it is no longer needed. |
Running notebooks with the `FakeKyoto` backend was occassionally taking a very long time (see Qiskit#1307 (comment)). We believe this is due to some notebooks taking advantage of the large number of qubits it supports. This PR switches to Qiskit's `GenericBackendV2`. This should be much faster and gives us more fine-grained control over the test backend properties.
Closes Qiskit#1279 Closes Qiskit#1331 I removed the "ignore warning" code from all relevant tutorials (in addition to VQE and CHSH) and ran them all locally on simulators to make sure we aren't getting warnings. I also added Requirements sections to VQE and CHSH. Working on re-generating the output for CHSH since it currently runs on a 27Q system, but I'm in the queue for 16 hrs. --------- Co-authored-by: Frank Harkins <frankharkins@hotmail.co.uk>
Closes #1279
Closes #1331
I removed the "ignore warning" code from all relevant tutorials (in addition to VQE and CHSH) and ran them all locally on simulators to make sure we aren't getting warnings. I also added Requirements sections to VQE and CHSH.
Working on re-generating the output for CHSH since it currently runs on a 27Q system, but I'm in the queue for 16 hrs.