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

Update experiment service #945

Merged
merged 22 commits into from
Jul 27, 2021
Merged

Update experiment service #945

merged 22 commits into from
Jul 27, 2021

Conversation

jyu00
Copy link
Collaborator

@jyu00 jyu00 commented May 25, 2021

Summary

This PR updates the experiment service to use ExperimentData, AnalysisResult, and ExperimentService classes in qiskit-terra. This allows the new qiskit-experiments package to create experiments without requiring access to IBM Quantum.

Details and comments

Closes #873, #854.

@jyu00 jyu00 added Changelog: API Change Include in the Changed section of the changelog on hold Can not fix yet labels May 25, 2021
@jyu00
Copy link
Collaborator Author

jyu00 commented May 25, 2021

This should not be merged until Qiskit/qiskit#5499 is merged

@jyu00 jyu00 added this to the 0.15 milestone Jun 22, 2021
@jyu00 jyu00 modified the milestones: 0.15, 0.16 Jun 28, 2021
@jyu00 jyu00 marked this pull request as draft July 1, 2021 21:24
jyu00 added 2 commits July 26, 2021 14:40
# Conflicts:
#	qiskit/providers/ibmq/experiment/experimentservice.py
@jyu00 jyu00 marked this pull request as ready for review July 27, 2021 13:32
@jyu00 jyu00 removed the on hold Can not fix yet label Jul 27, 2021
@jyu00 jyu00 merged commit 5c11f79 into Qiskit:master Jul 27, 2021
@jyu00 jyu00 deleted the experiment branch July 27, 2021 20:22
pip install -U -c constraints.txt -r requirements-dev.txt
pip install -c constraints.txt -e .
- name: Run Tests
run: make runtime_integration
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be experiment_integration?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I forgot to push my changes before merging it 🤦‍♀️

mriedem added a commit to mriedem/qiskit-ibmq-provider that referenced this pull request Jul 27, 2021
jyu00 pushed a commit that referenced this pull request Jul 28, 2021
@jyu00 jyu00 mentioned this pull request Jul 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog: API Change Include in the Changed section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make experiment.device_components() output more readable
2 participants