-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add get_partial_result #135
Conversation
It seems H1-2E isn't responding. Will rerun the tests |
:type handle: ResultHandle | ||
|
||
:return: A tuple containing the results and circuit status. | ||
If no results are available, the first element is None. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need an indent here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
I hope this should fix the issue with the version, it would be great if we would not need to update the ci workflow |
The CI failure is caused by the stabiliser simulator keeping non-Clifford circuits in a forever |
Websocket only message back on status change, therefore it is not useful for retrieving partial results. I decided to add a simple method that makes a single call to the endpoint. It's up to the user to decide how they want to use this method and process data.