-
Notifications
You must be signed in to change notification settings - Fork 92
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
Propagate Qiskit's circuit metadata to jobs for Azure Quantum backends #278
Propagate Qiskit's circuit metadata to jobs for Azure Quantum backends #278
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.
Can we add/modify some test that checks for this?
This PR should address #259 |
This is also a fix for running VQE, mentioned in #224 |
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.
The original changes only fixed IonQ backends. My latest push refactors the Qiskit backends to have a common base class that appends the metadata by default. It also removes a bunch of duplicated code.
The new common base Backend class addresses #263 |
addresses #258 |
Running StateTomography from qiskit_experiments requires information from the circuit metadata that should be present in the job output. If those properties are not present, analysis of results will fail.
This change adds the corresponding circuit metadata.