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

Incoherence in notation for RZX gate #12684

Closed
1 of 5 tasks
bowangle opened this issue Jun 28, 2024 · 2 comments · Fixed by #12754
Closed
1 of 5 tasks

Incoherence in notation for RZX gate #12684

bowangle opened this issue Jun 28, 2024 · 2 comments · Fixed by #12754
Assignees
Labels
documentation Something is not clear or an error documentation good first issue Good for newcomers

Comments

@bowangle
Copy link

bowangle commented Jun 28, 2024

URL to the relevant documentation

https://docs.quantum.ibm.com/api/qiskit/qiskit.circuit.library.RZXGate

In examples section, third equation

Select all that apply

  • typo
  • code bug
  • out-of-date content
  • broken link
  • other

Describe the fix.

If we follow the notation introduce for RZX(θ) q0,q1=exp(−i θ/2 X⊗Z) which are the default notation, it would mean that in the "examples" section, we should have RZX(θ=π)=−iX⊗Z instead of −iZ⊗X.
It's not coherent with the fact that in the note section, we use big-endian convention for the tensor notation.
Or else we have to tell in which way we apply the gate by adding the q1,q0 index like this: RZX(θ=π)q1,q0=−iZ⊗X.

@Eric-Arellano
Copy link
Collaborator

Thanks! This is an issue in the API docs, so it needs to be fixed in qiskit/qiskit. I'm transferring it there.

@Eric-Arellano Eric-Arellano transferred this issue from Qiskit/documentation Jun 28, 2024
@Eric-Arellano Eric-Arellano added the documentation Something is not clear or an error documentation label Jun 28, 2024
@1ucian0 1ucian0 added the good first issue Good for newcomers label Jul 5, 2024
@github-project-automation github-project-automation bot moved this to Tagged but unassigned in Contributor Monitoring Jul 5, 2024
@Ameya28Bhave
Copy link
Contributor

Hi @1ucian0 . Can you please assign me this issue. This would be helpful as my first qiskit contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Something is not clear or an error documentation good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants