-
Notifications
You must be signed in to change notification settings - Fork 984
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 PhysicalZTag #2753
Add PhysicalZTag #2753
Conversation
dstrain115
commented
Feb 10, 2020
- Simple class in cirq.google.ops that can denote physical Z gates.
- Just includes the class definition so far, no functionality.
- Simple class in cirq.google.ops that can denote physical Z gates. - Just includes the class definition so far, no functionality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments, then LGTM.
cirq/google/ops/physical_z_tag.py
Outdated
class PhysicalZTag: | ||
"""Class to add as a tag onto an Operation to denote a Physical Z operation. | ||
|
||
By default, all Z rotations on Google devices are considered to be virtual. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: 4-space indent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
cirq/google/ops/physical_z_tag.py
Outdated
tracking mechanisms, essentially commuting it forwards through the circuit | ||
until it hits a non-commuting operation (Such as a sqrt(iSwap)). | ||
|
||
By attaching this to a Z rotation operation, this class will denote to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: "When applied to a Z rotation operation, this tag indicates to the hardware that..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.