Skip to content

Implemented PVQD algorithm with primitives.#8705

Merged
mergify[bot] merged 61 commits into
Qiskit:mainfrom
dlasecki:pvqd-primitives
Sep 22, 2022
Merged

Implemented PVQD algorithm with primitives.#8705
mergify[bot] merged 61 commits into
Qiskit:mainfrom
dlasecki:pvqd-primitives

Conversation

@dlasecki
Copy link
Copy Markdown
Contributor

@dlasecki dlasecki commented Sep 8, 2022

Summary

Implements the Projected Variational Quantum Dynamics algorithm with primitives.
Removes the old algorithm.

Closes: #8489

Details and comments

This PR is blocked by #8681 and #8683 PRs and the current draft includes files from those PRs as they are necessary for the algorithm to run. Please ignore these files while reviewing this PR.

  • Resolve cyclic imports.

No deprecation needed for the old version because it was never released.

@woodsp-ibm woodsp-ibm added this to the 0.22 milestone Sep 12, 2022
@dlasecki dlasecki marked this pull request as ready for review September 13, 2022 19:55
@dlasecki dlasecki requested review from a team and manoelmarques as code owners September 13, 2022 19:55
Dariusz Lasecki added 2 commits September 15, 2022 13:28
@woodsp-ibm woodsp-ibm added the Changelog: Added Add an "Added" entry in the GitHub Release changelog. label Sep 16, 2022
Comment thread qiskit/algorithms/__init__.py Outdated
Comment thread qiskit/algorithms/time_evolvers/pvqd/pvqd.py Outdated
Comment thread qiskit/algorithms/time_evolvers/pvqd/pvqd.py Outdated
dlasecki and others added 3 commits September 22, 2022 09:07
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Copy link
Copy Markdown
Member

@woodsp-ibm woodsp-ibm left a comment

Choose a reason for hiding this comment

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

@dlasecki Thanks, LGTM now.

@mergify mergify Bot merged commit cde535c into Qiskit:main Sep 22, 2022
ElePT pushed a commit to ElePT/qiskit that referenced this pull request Jun 27, 2023
* Implemented observables_evaluator.py with primitives.

* Added evolvers problems and interfaces to time_evolvers package.

* Mostly updated trotter_qrte.py to use primitives.

* Added observables_evaluator.py that uses primitives.

* Added observables_evaluator.py that uses primitives.

* Updated trotter_qrte.py to use primitives.

* Updated imports

* Added estimator to pvqd.py (draft)

* Updated typehints and limited use of opflow.

* Updated typehints and limited use of opflow.

* Removed files out of scope for this PR.

* Added annotations import.

* Added observables_evaluator.py with primitives.

* Added ListOrDict support to observables_evaluator.py.

* Drafted pvqd.py with the fidelity primitive; type hints updated.

* Accepting fidelity primitive.

* Included CR suggestions.

* Applied some CR comments.

* Applied some CR comments.

* Added reno.

* Added reno.

* Accepting Statevector.

* Added attributes docs.

* Support for 0 operator.

* Implemented pvqd unit tests with primitives; code refactoring.

* Added reno.

* Updated algorithms init.

* Code refactoring.

* Removed old pvqd algorithm and related files.

* Add pending deprecation for evolvers

* Renamed classes and linked to algorithms init.

* fix docstring

* Improved reno.

* Updated classes names.

* Code refactoring.

* Applied CR comments.

* Removed unnecessary files.

* Applied CR comments.

* Black fix.

* Applied CR comments.

* Init updates.

* Reduced number of cyclic import.

* Fix for cyclic import

* Doc fix

* Updated init.

* Added error raising and unit test.

* Apply suggestions from code review

Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>

* Fixed code example.

Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com>
Co-authored-by: woodsp-ibm <woodsp@us.ibm.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
ElePT pushed a commit to ElePT/qiskit-algorithms-test that referenced this pull request Jul 17, 2023
* Implemented observables_evaluator.py with primitives.

* Added evolvers problems and interfaces to time_evolvers package.

* Mostly updated trotter_qrte.py to use primitives.

* Added observables_evaluator.py that uses primitives.

* Added observables_evaluator.py that uses primitives.

* Updated trotter_qrte.py to use primitives.

* Updated imports

* Added estimator to pvqd.py (draft)

* Updated typehints and limited use of opflow.

* Updated typehints and limited use of opflow.

* Removed files out of scope for this PR.

* Added annotations import.

* Added observables_evaluator.py with primitives.

* Added ListOrDict support to observables_evaluator.py.

* Drafted pvqd.py with the fidelity primitive; type hints updated.

* Accepting fidelity primitive.

* Included CR suggestions.

* Applied some CR comments.

* Applied some CR comments.

* Added reno.

* Added reno.

* Accepting Statevector.

* Added attributes docs.

* Support for 0 operator.

* Implemented pvqd unit tests with primitives; code refactoring.

* Added reno.

* Updated algorithms init.

* Code refactoring.

* Removed old pvqd algorithm and related files.

* Add pending deprecation for evolvers

* Renamed classes and linked to algorithms init.

* fix docstring

* Improved reno.

* Updated classes names.

* Code refactoring.

* Applied CR comments.

* Removed unnecessary files.

* Applied CR comments.

* Black fix.

* Applied CR comments.

* Init updates.

* Reduced number of cyclic import.

* Fix for cyclic import

* Doc fix

* Updated init.

* Added error raising and unit test.

* Apply suggestions from code review

Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>

* Fixed code example.

Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com>
Co-authored-by: woodsp-ibm <woodsp@us.ibm.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: Added Add an "Added" entry in the GitHub Release changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rewrite PVQD with primitives.

6 participants