-
Notifications
You must be signed in to change notification settings - Fork 45
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
Add support for verbatim circuits and disable_qubit_rewiring #61
Add support for verbatim circuits and disable_qubit_rewiring #61
Conversation
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.
@dexter2206 amazing work! All good on code side!
I only have comment on notebook. Last cell is not executed, so, we do not see result. And maybe we should catch error. By that we will show that error is expected.
What do you think?
I think we merge this and fix #45 and we will cut new release. |
Thanks! I will change the notebook as suggested tomorrow when Aspen is available. |
@dexter2206 hi there! any updates here? :) |
Summary
This PR implements support for verbatim boxes (only for whole circuits) and disabling qubit rewiring. The solution follows exactly the discussion in #58
Details and comments
Note that tests for the functionality have a
skip
mark because they necessarily have to access external resources.