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

Several "Truncating to maximum value" messages when test.python.providers #4635

Closed
1ucian0 opened this issue Jul 1, 2020 · 14 comments · Fixed by #5176
Closed

Several "Truncating to maximum value" messages when test.python.providers #4635

1ucian0 opened this issue Jul 1, 2020 · 14 comments · Fixed by #5176
Assignees
Labels
good first issue Good for newcomers priority: low type: enhancement It's working, but needs polishing

Comments

@1ucian0
Copy link
Member

1ucian0 commented Jul 1, 2020

When testing the providers, there are several warnings about "error parameter greater than maximum allowed value".

python -m unittest discover test.python.providers
.........................Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.066694 > 1.066667). Truncating to maximum value.
.Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.066694 > 1.066667). Truncating to maximum value.
.Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.066694 > 1.066667). Truncating to maximum value.
.Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.066694 > 1.066667). Truncating to maximum value.
.........................................................Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.084497 > 1.066667). Truncating to maximum value.
Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
[...]
Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.068462 > 1.066667). Truncating to maximum value.
Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.068117 > 1.066667). Truncating to maximum value.
Device reported a gate error parameter greater than maximum allowed value (1.000000 > 0.800000). Truncating to maximum value.
Device model returned a depolarizing error parameter greater than maximum allowed value (1.068273 > 1.066667). Truncating to maximum value.
.........s...ssss.sssssss....s.ss.................................................................

Not sure if those are expected. If so, they should be captured (and checked) by the test themselves.

@1ucian0 1ucian0 added type: enhancement It's working, but needs polishing good first issue Good for newcomers priority: low labels Jul 1, 2020
@rochisha0
Copy link
Contributor

rochisha0 commented Jul 1, 2020

What is to be done on this issue? @1ucian0

@1ucian0
Copy link
Member Author

1ucian0 commented Jul 1, 2020

It seems like Aer introduced this warnings in Qiskit/qiskit-aer#792
In the same commit, Aer is adding a parameter from_backend(..., warning=<bool>) (here) that we can probably use to silence these output.

If you need sync help, look for me on Slack or Gitter. Thanks!

@VANRao-Stack
Copy link

VANRao-Stack commented Jul 21, 2020

Is this issue still unaddressed, and if it's okay with everybody, could I give it a shot? @1ucian0 @rochisha0

@schefferac2020
Copy link

Is this an actual error or just warnings? Thanks

@rochisha0
Copy link
Contributor

@VANRao-Stack Yeah sure, from my side.

@coder-mtv
Copy link

Is someone working on it, or can I give it a shot?

@nonhermitian
Copy link
Contributor

There is nothing really to fix. This is just a warning that the calibration results from the devices are not within given physical ranges. This is to be expected because the value indicating an error is 1.0. As of Aer 0.6 these warnings can be disabled.

@clarebirch
Copy link

Would it be a good idea to modify so that it only returns the warning once, or so that the default is to have the warning off?

@clarebirch
Copy link

clarebirch commented Aug 14, 2020

Yeah, sure, but is that what we want to do? Seems like it'd be best to have the warnings upfront (e.g. for researchers' sake). That said, once a device model is reporting gate errors once, it'll be reporting them repeatedly (right?) which would be unnecessary + annoying. We could add a bool to stop the prints after the first?

@VANRao-Stack
Copy link

Is someone working on it, or can I give it a shot?

Yes please, do go ahead, sorry for the late reply, my exams are still going on. Cheers!

@1ucian0
Copy link
Member Author

1ucian0 commented Oct 1, 2020

@coder-mtv Want to be assigned?

@sagarpahwa
Copy link
Contributor

Hi @1ucian0, can I try this one?

@1ucian0
Copy link
Member Author

1ucian0 commented Oct 2, 2020

If @coder-mtv does not answer in the next 8 hours, you can take it @sagarpahwa

sagarpahwa added a commit to sagarpahwa/qiskit-terra that referenced this issue Oct 3, 2020
sagarpahwa added a commit to sagarpahwa/qiskit-terra that referenced this issue Oct 3, 2020
@mergify mergify bot closed this as completed in #5176 Oct 3, 2020
mergify bot pushed a commit that referenced this issue Oct 3, 2020
* fix #4635 by setting warnings=False for NoiseModel of FakeBackend

* retrigger the pipeline #4635

* Update qiskit/test/mock/fake_backend.py

Co-authored-by: Luciano Bello <luciano.bello@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers priority: low type: enhancement It's working, but needs polishing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants