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

Doc string for SQISW contains typos (unitary is incorrect) #1721

Open
steve-jeffrey opened this issue Jan 9, 2024 · 0 comments
Open

Doc string for SQISW contains typos (unitary is incorrect) #1721

steve-jeffrey opened this issue Jan 9, 2024 · 0 comments
Labels
bug 🐛 An issue that needs fixing.

Comments

@steve-jeffrey
Copy link

steve-jeffrey commented Jan 9, 2024

Pre-Report Checklist

  • [ X] I am running the latest versions of pyQuil and the Forest SDK
  • [ X] I checked to make sure that this bug has not already been reported

Issue Description

The doc string for the SQISW gate in pyquil.simulation.matrices has some issues:

  • the formatting is not correct,
  • elements are missing from the 3rd row of the unitary.

There is also a typo in the word "conditonal" in the phrase "XX+YY interaction with conditonal phase on |11>" which appears in the doc string for both the FSIM and PHASEDFSIM gates.

How to Reproduce

Check the source code or the documentation artifacts. The latter is the easiest way of seeing the problems (as the documentation is nicely formatted).

@steve-jeffrey steve-jeffrey added the bug 🐛 An issue that needs fixing. label Jan 9, 2024
@steve-jeffrey steve-jeffrey changed the title Doc string from SQISW contains typos (unitary is incorrect) Doc string for SQISW contains typos (unitary is incorrect) Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue that needs fixing.
Projects
None yet
Development

No branches or pull requests

1 participant