-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix #5497: update IonQ docs to remove decompose_operation #7198
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
Conversation
Removed mention of `decompose_operation`. Issue quantumlib#5497 talks about updating the IonQ documentation to describe the use of transformers for compilation introduced in quantumlib#5479. This commit does not address that.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7198 +/- ##
=======================================
Coverage 98.14% 98.14%
=======================================
Files 1100 1100
Lines 96147 96241 +94
=======================================
+ Hits 94365 94458 +93
- Misses 1782 1783 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Replace and test the outdated example code (cirq.Circuit does not take a `device` argument anymore).
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.
LGTM after fixing the example code. Please check if it looks OK.
I'm glad you reviewed that :-). |
…uantumlib#7198) * Fix quantumlib#5497 Removed mention of `decompose_operation`. Issue quantumlib#5497 talks about updating the IonQ documentation to describe the use of transformers for compilation introduced in quantumlib#5479. This commit does not address that. * Fix example of circuit transformation for IonQ-native gates Replace and test the outdated example code (cirq.Circuit does not take a `device` argument anymore). --------- Co-authored-by: Pavol Juhas <[email protected]>
Removed mention of
decompose_operation
.Issue #5497 talks about updating the IonQ documentation to describe the use of transformers for compilation introduced in #5479. This commit does not address that. I think it would be better to leave IonQ doc additions to a separate PR.