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

changed cryptic u gates to readable x and h #1301

Merged
merged 5 commits into from
Oct 2, 2020

Conversation

rochisha0
Copy link
Contributor

Summary

Fixes #1287

Changed cryptic looking u gates to readable x and h gates.
U(pi, 0, pi) --> X
U(pi/2, 0, pi) --> H

Details and comments

@Cryoris
Copy link
Contributor

Cryoris commented Oct 2, 2020

Nice! There are a few more we can update:

  • In reciprocals we have cu(2 * np.pi, 0, 0, 0, q0, q1) which is the same as cry(2 * np.pi, q0, q1)
  • In some places we have p(pi/4) which is the same as t and p(-pi/4) which is the same as tdg
  • Similarly p(pi/2) is s and p(-pi/2) is sdg

@rochisha0
Copy link
Contributor Author

Done @Cryoris !

Copy link
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.

Thanks for the contribution which makes reading the circuits easier.

@manoelmarques manoelmarques merged commit 4cca284 into qiskit-community:master Oct 2, 2020
mtreinish pushed a commit to mtreinish/qiskit-core that referenced this pull request Nov 20, 2020
…aqua#1301)

* changed cryptic u gates to readable x and h

* fix lint

* changed few more gates

Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
manoelmarques pushed a commit to manoelmarques/qiskit-terra that referenced this pull request Dec 2, 2020
…aqua#1301)

* changed cryptic u gates to readable x and h

* fix lint

* changed few more gates

Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace generic 1q gates with specific angles by more readable gate names
4 participants