-
Notifications
You must be signed in to change notification settings - Fork 368
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 to_instruction to ReadoutError #257
Add to_instruction to ReadoutError #257
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.
Good to go!
CHANGELOG.md
Outdated
@@ -39,6 +39,7 @@ Fixed | |||
|
|||
Added | |||
----- | |||
- Added `to_instruction` method to `ReadoutError`. |
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.
- Added `to_instruction` method to `ReadoutError`. | |
- Added `to_instruction` method to `ReadoutError`. (#257) |
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.
(postfix the PR number)
src/framework/operations.hpp
Outdated
@@ -771,7 +771,8 @@ Op json_to_op_roerror(const json_t &js) { | |||
op.name = "roerror"; | |||
JSON::get_value(op.memory, "memory", js); | |||
JSON::get_value(op.registers, "register", js); | |||
JSON::get_value(op.probs, "probabilities", js); | |||
JSON::get_value(op.probs, "probabilities", js); // DEPRECIATED: Remove in 0.4 |
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.
Pretty sure you'll know better than me, but I'd swear that "depreciated" is used in financial-related context. it's more common to use "deprecate" here :)
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.
It's value has gone down since we included it
Summary
Adds missing
to_instruction
method toReadoutError
object. This allows directly inserting readout errors into aQuantumCircuit
for simulation inAer
like withQuantumError
objects which can be directly inserted into circuits as"kraus"
instructions.Details and comments
Example: 2-qubit readout error