-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Improve performance of BackendSamplerV2 and EstimatorV2 #12291
Conversation
Pull Request Test Coverage Report for Build 8802706774Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
One or more of the the following people are requested to review this:
|
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 for this quick work. I've left a few comments, which I mostly made as I read through the file figuring out what was going on. It all makes sense to me, so LGTM!
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
I refactored BackendEstimatorV2 by introducing an internal data class |
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
Summary
Fixes #12290
Addresses Qiskit/qiskit-ibm-runtime#1631 (need to port this PR to qiskit-ibm-runtime to close it)
TODO
Details and comments
Sampler
Estimator