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

Rename shift and clock matrices to gen_pauli_x and gen_pauli_z #478

Closed
vprusso opened this issue Feb 22, 2024 · 5 comments
Closed

Rename shift and clock matrices to gen_pauli_x and gen_pauli_z #478

vprusso opened this issue Feb 22, 2024 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Milestone

Comments

@vprusso
Copy link
Owner

vprusso commented Feb 22, 2024

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.

@vprusso vprusso added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Feb 22, 2024
@MohitKambli
Copy link
Contributor

Hello @vprusso ,
Hope you're doing well.
I would like to work on this issue.
Just wanted to know one thing.
According to the requirement, we need to change the filenames, method names, method calls and comments (shift to gen_pauli_x and clock to gen_pauli_z).
Is that correct or did I misunderstand anything?
Kindly let me know if there's something else to it.

Thanks and Regards,
Mohit Kambli

@vprusso
Copy link
Owner Author

vprusso commented Feb 22, 2024

Hi @MohitKambli . Thank you for the interest in contributing to toqito!

Indeed, you got it quite right--it's simply a matter of altering the names from shift to gen_pauli_x and clock to gen_pauli_z in the appropriate locations within the project. Should be a fairly slamdunk PR I think.

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!

@MohitKambli
Copy link
Contributor

Hello @vprusso,
Thank you for clarifying my concerns.
I have made the potential changes to solve this issue and raised a pull request for the same.
If at all there are any changes/inputs required from my end, then do let me know.
Sincere apologies if I have implemented the code changes incorrectly or incompletely.

Thanks and Regards,
Mohit Kambli

@vprusso
Copy link
Owner Author

vprusso commented Feb 23, 2024

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!

MohitKambli added a commit to MohitKambli/toqito that referenced this issue Feb 23, 2024
@MohitKambli
Copy link
Contributor

Hello @vprusso,
Hope you're doing well.
Yes, I did go through @purva-thakre's comments, made changes and proceeded accordingly.
Sincere apologies for the mistakes coming from my end.
If at all there any more changes required, then do let me know.
Thank you for understanding my part.
Hope this commit meets the requirement.

Thanks and Regards,
Mohit Kambli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants