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

Allow importing SamplerV1 and EstimatorV1 #1477

Merged
merged 3 commits into from
Mar 6, 2024

Conversation

Eric-Arellano
Copy link
Collaborator

Right now, users can't import qiskit_ibm_runtime.SamplerV1 and qiskit_ibm_runtime.EstimatorV1 because we only expose the aliases Sampler and Estimator. That gets in the way of users being explicit about which primitive version they're using.

This also is resulting in the API docs not including the pages for both the V1 and V2 primitives in Qiskit/documentation#973 because we didn't tell autosummary about them.

@Eric-Arellano Eric-Arellano added the backport potential The bug might be minimal and/or important enough to be port to stable label Mar 6, 2024
@Eric-Arellano Eric-Arellano requested review from kt474 and jyu00 March 6, 2024 15:46
@Eric-Arellano
Copy link
Collaborator Author

@kt474 @jyu00 note that this is an actual user API change. You may want to do a patch release today with this fix.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 8175005887

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.284%

Totals Coverage Status
Change from base Build 8166472314: 0.0%
Covered Lines: 5764
Relevant Lines: 7005

💛 - Coveralls

@Eric-Arellano Eric-Arellano merged commit 4632a06 into Qiskit:main Mar 6, 2024
20 checks passed
@Eric-Arellano Eric-Arellano deleted the EA/expose-v1-vs-v2 branch March 6, 2024 16:30
Eric-Arellano added a commit to Eric-Arellano/qiskit-ibm-runtime that referenced this pull request Mar 6, 2024
Right now, users can't import `qiskit_ibm_runtime.SamplerV1` and `qiskit_ibm_runtime.EstimatorV1` because we only expose the aliases `Sampler` and `Estimator`. That gets in the way of users being explicit about which primitive version they're using.

This also is resulting in the API docs not including the pages for both the V1 and V2 primitives in Qiskit/documentation#973 because we didn't tell autosummary about them.
kt474 pushed a commit that referenced this pull request Mar 6, 2024
Right now, users can't import `qiskit_ibm_runtime.SamplerV1` and `qiskit_ibm_runtime.EstimatorV1` because we only expose the aliases `Sampler` and `Estimator`. That gets in the way of users being explicit about which primitive version they're using.

This also is resulting in the API docs not including the pages for both the V1 and V2 primitives in Qiskit/documentation#973 because we didn't tell autosummary about them.
@kt474 kt474 added the Changelog: Bugfix Include in the Fixed section of the changelog label Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport potential The bug might be minimal and/or important enough to be port to stable Changelog: Bugfix Include in the Fixed section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants