qiskit-community / qiskit-ignis

Ignis (deprecated) provides tools for quantum hardware verification, noise characterization, and error correction.
Apache License 2.0
168 stars 162 forks source link

Updated analysis.py to fix for issue#511(Wrong arguments) #593

Open jessica-angel7 opened 2 years ago

jessica-angel7 commented 2 years ago

Summary

Fix for issue#511. The issue mentions that the error mitigated fidelity should be composed of P0, P1, and C, not P0_m, P1_m, and C. But in line 192, for the Fourier Analysis, we don't have to apply the mitigated values for the first term. Hence the changes have been made. The arguments P0_m and P1_m have been changed to P0 and P1.

Details and comments

This updates have been made in the understanding that issue has a valid point. This is my first contribution for this project. Please Review and let me know if am wrong.