-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
latex circuit drawer needs a consistent visualization of cu1, crz, rzz #1957
Labels
good first issue
Good for newcomers
Comments
#1960 fixes the text drawer. |
At this point, this issue only affects the latex drawer. Renaming. |
@1ucian0 I am going to try to deep on this issue . OK? |
Sure! Let me know if you need any pointers!
…On Fri, Aug 2, 2019, 3:08 PM Elisa Martin ***@***.***> wrote:
@1ucian0 <https://github.com/1ucian0> I am going to try to deep on this
issue . OK?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1957>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAF3FZKV7MXJP3ZYTCA4R3TQCSA2ZANCNFSM4G5H4D4A>
.
|
any progress @ElisaGarijo ? Let me know! |
brandhsn
added a commit
to brandhsn/qiskit-terra
that referenced
this issue
Sep 26, 2019
brandhsn
added a commit
to brandhsn/qiskit-terra
that referenced
this issue
Sep 26, 2019
1ucian0
pushed a commit
that referenced
this issue
Sep 30, 2019
* Modified rzz visualization in latex circuit drawer to match the text/mpl drawer (#1957) * added release note * small release note fix
faisaldebouni
pushed a commit
to faisaldebouni/qiskit-terra
that referenced
this issue
Aug 5, 2020
…it#3155) * Modified rzz visualization in latex circuit drawer to match the text/mpl drawer (Qiskit#1957) * added release note * small release note fix
lcapelluto
pushed a commit
to lcapelluto/qiskit-terra
that referenced
this issue
Sep 17, 2020
…it#3155) * Modified rzz visualization in latex circuit drawer to match the text/mpl drawer (Qiskit#1957) * added release note * small release note fix
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What is the expected enhancement?
Let's make sure the text, mpl and latex drawer have a consistent representation for these.
There are some bugs, for example the
latex
doesn't draw rzz at all.And there are some inconsistencies, for example
mpl
draws rzz as a line andtext
draws it as a box.After discussions with @jaygambetta, let's do it like this:
This just signifies that the 1st and 3rd gates are symmetric gates.
The text was updated successfully, but these errors were encountered: