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

Add QPY serialization for PauliEvolutionGate (backport #7374) #7382

Merged
merged 1 commit into from
Dec 8, 2021

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 8, 2021

This is an automatic backport of pull request #7374 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* WIP: Add qpy serialization for PauliEvolutionGate

This commit adds serialization for the PauliEvolutionGate class so that
we can exactly reproduce a PauliEvolutionGate over QPY. This works by
bumping the qpy format version and adding new structs to represent the
PauliEvolutionGate and all it's child attribute types.

* Fix handling of operator in PauliEvolutionGate

This commit fixes the handling of the operator attribute in the
PauliEvolutionGate. With this commit we can serialize a
PauliEvolutionGate correctly with the exception of it's synthesis class
which still needs to be supported.

* settings for synth

* Add support for custom synthesis classes

* Expand test coverage

* Add release notes

* Update release note

* fix param binding in PauliEvo

* allow time as an int

* Apply suggestions from code review

Co-authored-by: Jake Lishman <jake@binhbar.com>

* Update qiskit/circuit/qpy_serialization.py

Co-authored-by: Jake Lishman <jake@binhbar.com>

* Update qiskit/circuit/qpy_serialization.py

* Rerun with latest black

* Close buffers when finished

* Fix release note wording

* Fix lint

* Adjust tests around extra layer of gates

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Jake Lishman <jake@binhbar.com>
(cherry picked from commit 9af5ca1)
@mergify mergify bot requested review from manoelmarques, woodsp-ibm and a team as code owners December 8, 2021 21:44
@jakelishman jakelishman added automerge Changelog: Bugfix Include in the "Fixed" section of the changelog labels Dec 8, 2021
@jakelishman jakelishman added this to the 0.19.1 milestone Dec 8, 2021
@mergify mergify bot merged commit 73024df into stable/0.19 Dec 8, 2021
@mergify mergify bot deleted the mergify/bp/stable/0.19/pr-7374 branch December 8, 2021 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the "Fixed" section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants