Skip to content

Conversation

@jan-an
Copy link
Collaborator

@jan-an jan-an commented Dec 23, 2025

Copy link
Collaborator

@yaelbh yaelbh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jan-an.
Note that another bug will arise if you run backend with use_fractional_gates=True, for a backend that has only rx and no rzz, then afterwards call backend again with the same backend name but use_fractional=False - a cached configuration will be taken instead of a refreshed one (this is the line). Similarly, the line before will cause the configuration to be refreshed again and again for repeated calls to backend with use_fractional=True, this is not a functional bug but a performance one. I'll open an issue about these bugs.

@jan-an
Copy link
Collaborator Author

jan-an commented Dec 24, 2025

Thanks @jan-an. Note that another bug will arise if you run backend with use_fractional_gates=True, for a backend that has only rx and no rzz, then afterwards call backend again with the same backend name but use_fractional=False - a cached configuration will be taken instead of a refreshed one (this is the line). Similarly, the line before will cause the configuration to be refreshed again and again for repeated calls to backend with use_fractional=True, this is not a functional bug but a performance one. I'll open an issue about these bugs.

Hi @yaelbh, thanks a lot for pointing out this problem! Would you like me to address anything related to that as part of this PR?

@yaelbh
Copy link
Collaborator

yaelbh commented Dec 24, 2025

This PR can be merged, anything else can be done in other PRs.

@jan-an jan-an added this pull request to the merge queue Dec 24, 2025
Merged via the queue into Qiskit:main with commit 3615b31 Dec 24, 2025
17 checks passed
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

Successfully merging this pull request may close these issues.

QiskitRuntimeService should also check for rx in the backend basis when fractional gates=True

2 participants