Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

Revert #1440 i.e. remove PauliSumOp #1542

Closed
woodsp-ibm opened this issue Feb 23, 2021 · 0 comments · Fixed by #1544
Closed

Revert #1440 i.e. remove PauliSumOp #1542

woodsp-ibm opened this issue Feb 23, 2021 · 0 comments · Fixed by #1544
Assignees

Comments

@woodsp-ibm
Copy link
Member

A PauliSumOp was introduced by #1440 however with the move of opflow to qiskit-terra the work around PauliSumOp functionality was completed there. The function added here has not been released, and the code was moved and is now updated, maintained in terra repo. It is not explicitly used in Aqua outside of opflow other than unit tests with the exception of being added into the QP convertor from_ising method. As this was the first version, and has been improved/fixed now in terra where it was moved, and has never been released in Aqua, I think its better not to introduce this at the time we are deprecating the repo so am proposing that this PR be reverted to remove PauliSumOp.

Any thoughts to the contrary @ikkoham @t-imamichi @Cryoris @stefan-woerner ? I will note that the PR would need to be reverted manually as files have changed since it was merged such that it cannot automatically create a PR that would revert the changes back again.

@ikkoham ikkoham self-assigned this Feb 24, 2021
@ikkoham ikkoham mentioned this issue Feb 24, 2021
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 a pull request may close this issue.

2 participants