-
Notifications
You must be signed in to change notification settings - Fork 586
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 error mitigation batch transform #1813
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1813 +/- ##
=======================================
Coverage 98.96% 98.96%
=======================================
Files 217 218 +1
Lines 16337 16368 +31
=======================================
+ Hits 16168 16199 +31
Misses 169 169
Continue to review full report at Codecov.
|
…to mitigate_transform
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.
Thanks @trbromley! Have left comments and suggestions, but a very nice PR
The unmitigated circuit result is ``0.33652776`` while the ideal circuit result is | ||
``0.23688169`` and we can hence see that mitigation has helped reduce our estimation error. |
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.
not required at all for merging! but it would be nice to see the unmitigated execution and ideal execution somewhere 😆 Note here though, would be too much for the entry example. Maybe a bigger example inside UsageDetails
.
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.
🤔 How about we provide a link to the demo (when available)?
Co-authored-by: Josh Izaac <josh146@gmail.com>
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.
Thanks @josh146
The unmitigated circuit result is ``0.33652776`` while the ideal circuit result is | ||
``0.23688169`` and we can hence see that mitigation has helped reduce our estimation error. |
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.
🤔 How about we provide a link to the demo (when available)?
Context:
Adds a
mitigate_with_zne
tape transform to implement error mitigation using ZNE and unitary folding.Description of the Change:
Adds
transforms.mitigate.mitigate_with_zne
along with documentation and tests.Benefits:
Possible Drawbacks: