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

Deprecate PauliList estimator observables #11520

Merged
merged 5 commits into from
Jan 23, 2024

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Jan 9, 2024

Ports #11055 to stable/0.46, as it was not included in 0.45

Also, includes a docstring note for warning about it.

1ucian0 and others added 3 commits January 9, 2024 13:42
* Deprecate PauliList estimator observables

Deprecates using a `PauliList` as an observable that is implicitly converted to a `SparsePauliOp` with coefficients 1 when calling `Estimator.run`. Users should instead explicitly convert the argument using `SparsePauliOp(pauli_list)` first.

* Revert algorithms_test_case.py

* Update test/python/algorithms/algorithms_test_case.py

Co-authored-by: Takashi Imamichi <[email protected]>

* Update test/python/primitives/test_estimator.py

---------

Co-authored-by: Takashi Imamichi <[email protected]>
Co-authored-by: ikkoham <[email protected]>
Co-authored-by: Christopher J. Wood <[email protected]>
@1ucian0 1ucian0 added the Changelog: Deprecation Include in "Deprecated" section of changelog label Jan 9, 2024
@1ucian0 1ucian0 added this to the 0.46.0 milestone Jan 9, 2024
@1ucian0 1ucian0 requested review from ikkoham, t-imamichi and a team as code owners January 9, 2024 12:46
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core
  • @ajavadia
  • @ikkoham
  • @levbishop
  • @t-imamichi

Copy link
Member

@t-imamichi t-imamichi left a comment

Choose a reason for hiding this comment

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

LGTM

@ikkoham ikkoham added this pull request to the merge queue Jan 23, 2024
Merged via the queue into Qiskit:stable/0.46 with commit 1c4a4de Jan 23, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Deprecation Include in "Deprecated" section of changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants