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

Bug: QPU.execution_options not used for QPU.retrieve_results #1693

Closed
kalzoo opened this issue Nov 15, 2023 · 0 comments · Fixed by #1694
Closed

Bug: QPU.execution_options not used for QPU.retrieve_results #1693

kalzoo opened this issue Nov 15, 2023 · 0 comments · Fixed by #1694
Assignees
Labels
bug 🐛 An issue that needs fixing.

Comments

@kalzoo
Copy link
Contributor

kalzoo commented Nov 15, 2023

QPU allows execution_options to be set on both the class instance itself (self.execution_options) or on the .execute request(execution_options arg). It follows the right precedence order when calling submit, but does not consider self.execution_options in the construction of QPUExecuteResponse (here).

One effect of this: if get_qc() is invoked with an endpoint_id, that endpoint is not used for results retrieval, and pyQuil tries to use the default instead (gateway). If no such gateway exists, then results retrieval fails.

This can be quickly fixed, and should be covered with a new test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment