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

The probability solved in Dynamicsbackend exceeds 1. #237

Closed
to24toro opened this issue Jul 4, 2023 · 0 comments · Fixed by #239
Closed

The probability solved in Dynamicsbackend exceeds 1. #237

to24toro opened this issue Jul 4, 2023 · 0 comments · Fixed by #239
Labels
bug Something isn't working

Comments

@to24toro
Copy link
Contributor

to24toro commented Jul 4, 2023

Informations

What is the current behavior?

Dynamicsbackend is utilized for a pulse simulation and useful for the simulation of the calibration.
When we pull the above PR and simulate the rough amplitude calibration of a cross resonance gate, we have the probability dynamics backend calculated exceeds 1.

Suggested solutions

This error raises because there may be a rounding error in the way the probability is calculated and the deviation from one is too large to surpress an error when rng.choice is called.
We need to fix the way of normalizing stetes (https://github.com/Qiskit-Extensions/qiskit-dynamics/blob/37d12ac33fffd57daea3c9d5f9cda3f00acc37ea/qiskit_dynamics/backend/dynamics_backend.py#L795)
We could fix it to normalize the probabilities instead of normalizing the state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant