search

Qiskit / qiskit-ibm-runtime

IBM Client for Qiskit Runtime
https://docs.quantum.ibm.com/api/qiskit-ibm-runtime
Apache License 2.0
157 stars 156 forks source link

The tests issue many warnings #1910

Open yaelbh opened 2 months ago

yaelbh commented 2 months ago

Take for example a unit tests from CI of a random PR: https://github.com/Qiskit/qiskit-ibm-runtime/actions/runs/10744984346/job/29803163084?pr=1909.

yaelbh commented 2 months ago

In addition to warnings, there are hundreds of line with the following template:

backend_converter.convert_to_target:INFO:2024-09-06 20:50:58,037: Gate calibration for instruction u1 on qubits (0,) is found in the PulseDefaults payload. However, this entry is not defined in the gate mapping of Target. This calibration is ignored.
kt474 commented 2 months ago

In addition to warnings, there are hundreds of line with the following template:

backend_converter.convert_to_target:INFO:2024-09-06 20:50:58,037: Gate calibration for instruction u1 on qubits (0,) is found in the PulseDefaults payload. However, this entry is not defined in the gate mapping of Target. This calibration is ignored.

@t-imamichi Do you think this log (added in #1600) can be removed or suppressed?

t-imamichi commented 2 months ago

I copied the part from qiskit. You can suppress it by changing it from info to debug for example. https://github.com/Qiskit/qiskit/blob/2ef371ae0d159a6dfd643805f3e5e5fdec37ab88/qiskit/providers/backend_compat.py#L254-L260

Btw, I found that convert_to_target of qiskit was updated https://github.com/Qiskit/qiskit/pull/11996. It would be nice to sync with it.

It would be ideal if we can use convert_to_target of qiskit. But it does not still include reset as required gate. So we need to keep our own convert_to_target. I suggest discussing it with qiskit core team. https://github.com/Qiskit/qiskit/blob/2ef371ae0d159a6dfd643805f3e5e5fdec37ab88/qiskit/providers/backend_compat.py#L71