-
Notifications
You must be signed in to change notification settings - Fork 122
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
[WIP] Basic QST experiment class example #7
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.
I know that this is not for merging, but want to point out the fix nevertheless. Also, in the notebook, replace cell 10 by:
for i in range(3):
print(par_data.component_experiment_data(i), '\n')
def _run_analysis(self, experiment_data, **params): | ||
# Options | ||
standard_weights = params.pop("standard_weights", True) | ||
beta = params.pop("beta", 0.5) |
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.
Usage of pop
should be replaced by get
initial state circuit. If a Gate or Operator it will be | ||
appended to a quantum circuit. | ||
qubits (int or list or None): Optional, the qubits to be measured. | ||
If None all qubits will be measured. |
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.
basis
missing from docstring
Closing this since I started an PR for the actual tomography experiment |
Summary
This PR isn't intended for merging, but rather as a basic implementation of an experiment subclass.
Here a simple Pauli basis measurement quantum state tomography experiment is implemented. It depends on ignis, and is just a wrapper for internal tomography functions in ignis.
See the example notebook for demonstration.
Details and comments