Fix equality typos in Documentation for Ryy, Rxx and Rzx gates

[jofrevalles]

Summary

This PR resolves #13225 by fixing the typos in the documentation for the Ryy, Rxx and Rzx gates.

Details and comments

If I am not mistaken, these are the proper equalities. I also checked the result with numpy:

>>> import qiskit
>>> import numpy as np
>>> Ryy = qiskit.circuit.library.standard_gates.RYYGate(theta=np.pi)
>>> np.isclose(np.array(Ryy), -1j*np.kron(np.array(Y), np.array(Y)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])
>>> Rxx = qiskit.circuit.library.standard_gates.RXXGate(theta=np.pi)
>>> np.isclose(np.array(Rxx), -1j*np.kron(np.array(X), np.array(X)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])
>>> Rzx = qiskit.circuit.library.standard_gates.RZXGate(theta=np.pi)
>>> np.isclose(np.array(Rzx), -1j*np.kron(np.array(X), np.array(Z)))
array([[ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True],
       [ True,  True,  True,  True]])

[qiskit-bot]

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the following people are relevant to this code:

• @Cryoris
• @Qiskit/terra-core
• @ajavadia

[CLAassistant]

All committers have signed the CLA.

[Cryoris]

Thanks for the PR @jofrevalles, the fix looks good! The test failures seem unrelated, that shouldn't be triggered by a change in the docstring.

To continue with the PR, could you sign the CLA in the message above? 🙂

[jofrevalles]

Thanks for the PR @jofrevalles, the fix looks good! The test failures seem unrelated, that shouldn't be triggered by a change in the docstring.

To continue with the PR, could you sign the CLA in the message above? 🙂

Done! Thanks @Cryoris

[jakelishman]

Thanks!

[coveralls]

Pull Request Test Coverage Report for Build 11055286592

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 4 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.2%) to 88.988%

---

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/lex.rs	4	93.23%

Totals
Change from base Build 11052352075:	0.2%
Covered Lines:	55314
Relevant Lines:	62159

---

💛 - Coveralls