Skip to content

Add support for terra diagonal gate#683

Merged
chriseclectic merged 2 commits into
Qiskit:masterfrom
chriseclectic:fix/diagonal-gate
Apr 8, 2020
Merged

Add support for terra diagonal gate#683
chriseclectic merged 2 commits into
Qiskit:masterfrom
chriseclectic:fix/diagonal-gate

Conversation

@chriseclectic
Copy link
Copy Markdown
Member

Summary

Adds explicit support for terras diagonal gate operation.

Closes #682

Details and comments

This was previously internally supported through the unitary instruction but now adds an explicit diagonal_matrix op type for diagonal unitaries.

@chriseclectic chriseclectic added this to the Aer 0.4.2 milestone Apr 8, 2020
@chriseclectic chriseclectic requested a review from atilag as a code owner April 8, 2020 20:09
Copy link
Copy Markdown
Member

@atilag atilag left a comment

Choose a reason for hiding this comment

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

LGTM!

@chriseclectic chriseclectic merged commit 81ef953 into Qiskit:master Apr 8, 2020
hhorii pushed a commit to hhorii/qiskit-aer that referenced this pull request Apr 9, 2020
* Add loading of "diagonal" qobj instruction as diagonal "unitary"

* Add tests for diagonal gate
mtreinish pushed a commit to mtreinish/qiskit-aer that referenced this pull request Apr 9, 2020
* Add loading of "diagonal" qobj instruction as diagonal "unitary"

* Add tests for diagonal gate

(cherry picked from commit 81ef953)
@mtreinish mtreinish added the Changelog: Bugfix Include in the Fixed section of the changelog label Apr 9, 2020
@chriseclectic chriseclectic deleted the fix/diagonal-gate branch April 28, 2020 19:19
@chriseclectic chriseclectic added Changelog: New Feature Include in the Added section of the changelog and removed Changelog: Bugfix Include in the Fixed section of the changelog labels Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: New Feature Include in the Added section of the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for terra diagonal gate

3 participants