qiskit-tutorials icon indicating copy to clipboard operation
qiskit-tutorials copied to clipboard

Page "qiskit.providers.aer.noise.phase_damping_error" text copied from "qiskit.providers.aer.noise.phase_amplitude_damping_error"

Open gypsyzz opened this issue 3 years ago • 5 comments

The page "qiskit.providers.aer.noise.phase_damping_error" have description "Return a single-qubit combined phase and amplitude damping quantum error channel." even though it should only create phase damping error.

gypsyzz avatar Jan 27 '21 09:01 gypsyzz

Hi @gypsyzz this bug is fixed now. Please see https://github.com/Qiskit/qiskit-tutorials/blob/master/tutorials/simulators/3_building_noise_models.ipynb

@nonhermitian This issue can be closed

clausia avatar Jul 13 '21 08:07 clausia

The issue seems not to be fixed as this two pages are the same: https://qiskit.org/documentation/stubs/qiskit.providers.aer.noise.phase_amplitude_damping_error.html#qiskit.providers.aer.noise.phase_amplitude_damping_error https://qiskit.org/documentation/stubs/qiskit.providers.aer.noise.phase_damping_error.html At leaste the description of the function. I can arrange the buf if I am assigned the issue.

VicentePerezSoloviev avatar Oct 07 '21 12:10 VicentePerezSoloviev

I think this issue should be in the qiskit-aer repository, since the message is in the code documentation, please see:

  • https://github.com/Qiskit/qiskit-aer/blob/main/qiskit/providers/aer/noise/errors/standard_errors.py#L480
  • https://github.com/Qiskit/qiskit-aer/blob/main/qiskit/providers/aer/noise/errors/standard_errors.py#L593

clausia avatar Oct 07 '21 16:10 clausia

Yes, that is true. I will open an issue in qiskit-aer

VicentePerezSoloviev avatar Oct 07 '21 16:10 VicentePerezSoloviev

It is now fixed and pending to be accepted in https://github.com/Qiskit/qiskit-aer/pull/1374 .

VicentePerezSoloviev avatar Oct 07 '21 16:10 VicentePerezSoloviev

Looks like this was fixed in Aer. Thanks @VicentePerezSoloviev!

Eric-Arellano avatar Aug 18 '23 18:08 Eric-Arellano