Skip to content
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

Fix device noise model from BackendV2 not to have excessive quantum errors on measures/resets #1649

Merged
merged 5 commits into from
Nov 30, 2022

Conversation

itoko
Copy link
Contributor

@itoko itoko commented Nov 7, 2022

Summary

Fixes #1648 by updating device noise models not to have excessive quantum errors on non-gate instructions such as measures and resets when creating them from BackendV2.

Details and comments

This commit fixes qiskit_aer.noise.device.models.basic_device_gate_errors not to generate QuantumErrors (neither depolarizing nor relaxation) for Measure instruction and not to generate depolarizing errors for other non-Gate instructions when Target object is supplied.

@itoko itoko added stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable Changelog: Bugfix Include in the Fixed section of the changelog labels Nov 7, 2022
@itoko itoko added this to the Aer 0.11.2 milestone Nov 7, 2022
@itoko itoko force-pushed the fix-device-noise-from-target branch from f8378c1 to bda43f9 Compare November 7, 2022 07:26
@itoko itoko added the on hold Can not fix yet label Nov 7, 2022
@itoko
Copy link
Contributor Author

itoko commented Nov 7, 2022

I might need to rethink relaxation errors on resets (on hold until how to fix the spec).
-> I decided to stick to the same spec as device noise model from backend v1:

  • Resets may have relaxation errors but never have gate (depolar) errors
  • Measures never have quantum errors (neither gate errors nor relaxation errors) since they have readout errors

@itoko itoko added the automerge This PR will automatically merge once its CI has passed label Nov 30, 2022
@mergify mergify bot merged commit eab7d89 into Qiskit:main Nov 30, 2022
@itoko itoko deleted the fix-device-noise-from-target branch November 30, 2022 04:28
hhorii pushed a commit to hhorii/qiskit-aer that referenced this pull request Dec 1, 2022
…rrors on measures/resets (Qiskit#1649)

* Fix devide noise model from target

* Add release note

* Upgrade device noise models not to have errors on resets

* Revert the previous upgrade and fix the bug so that the behevior is the same as for backend v1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge This PR will automatically merge once its CI has passed Changelog: Bugfix Include in the Fixed section of the changelog stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Noise model created from BackendV2 contains excessive noises on measures.
2 participants