-
Notifications
You must be signed in to change notification settings - Fork 56
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
Rename shift and clock matrices to gen_pauli_x and gen_pauli_z #478
Comments
Hello @vprusso , Thanks and Regards, |
Hi @MohitKambli . Thank you for the interest in contributing to Indeed, you got it quite right--it's simply a matter of altering the names from The nice benefit is that doing this will serve as a nice exercise in familiarizing yourself with the project. Would welcome additional contributions, feedback, etc. from you as well! Looking forward to your contribution, and thank you again for your interest! |
Hello @vprusso, Thanks and Regards, |
Thanks again, @MohitKambli. You can refer to some of the comments that @purva-thakre made on the PR. Let us know if that is unclear, and thank you again! |
…gen_pauli_z (Updated Version V1)
Hello @vprusso, Thanks and Regards, |
The terminology for the "shift" and "clock" matrices is not necessarily well known in the quantum computing community, and in the context of quantum information, these matrices are more often referred to as the "generalized Pauli-X" and "generalized Pauli-Z" matrices. We should rename these.
We can still reference the matrices as is done in the functions, but we will want to explicitly point out that the generalized Pauli-X (shift) operator and generalized Pauli-Z (phase) operators are named as such.
The text was updated successfully, but these errors were encountered: