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 tests issue many warnings #1910

Open
yaelbh opened this issue Sep 8, 2024 · 3 comments
Open

The tests issue many warnings #1910

yaelbh opened this issue Sep 8, 2024 · 3 comments
Assignees

Comments

@yaelbh
Copy link
Collaborator

yaelbh commented Sep 8, 2024

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
Copy link
Collaborator Author

yaelbh commented Sep 8, 2024

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
Copy link
Member

kt474 commented Sep 9, 2024

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
Copy link
Member

t-imamichi commented Sep 10, 2024

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 Qiskit/qiskit#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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants