Skip to content
This repository was archived by the owner on Jul 28, 2023. It is now read-only.

Managed results #407

Merged
merged 13 commits into from
Oct 30, 2019
Merged

Managed results #407

merged 13 commits into from
Oct 30, 2019

Conversation

jyu00
Copy link
Collaborator

@jyu00 jyu00 commented Oct 23, 2019

Summary

Addresses #333. This is a follow up to #389 and #400.

Introduces a ManagedResults class that have the same methods (get_memory(), get_counts(), etc) as Result. It allows the user to do

job_set = job_manager.run()
result = job_set.results()
result.get_counts(1)

Details and comments


Returns:
Result object
Result object or ``None`` if result could not be retrieved.
Copy link
Member

Choose a reason for hiding this comment

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

Not blocking, but in the same spirit as the IBMQJob.result() discussion in an earlier PR, I wonder if it would not be too costly (in terms of implications and changes) to avoid having this method return None, for preserving the parallelism between a managed job and a IBMQJob.

@diego-plan9 diego-plan9 merged commit 2216e75 into Qiskit:master Oct 30, 2019
@jyu00 jyu00 deleted the result_manager branch November 21, 2019 19:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants