-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Support common notations in circuit_to_latex_using_qcircuit #4685
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
Comments
I would like to attempt working on this, please. |
I am still very very keen to work on this issue. Cheers, |
@jay-tau I've assigned it to you. |
Hello, |
Hello @vtomole, @pavoljuhas, @fedimser I would like to work on this issue. Could someone please assign it to me. Initial Completion Estimate ETA: 2/28/25 |
I was assigned this issue yesterday; therefore, I am pushing back my eta. Completion Estimate ETA: 3/7/25 |
@fedimser is there a particular name for the diagram that contains all of the gates you've listed to be reformatted? For naming the testcase. |
Successfully recreated desired circuit in testcase, does this look accurate @fedimser? Will now begin work on Spacing discrepancies addressed with the postprocessing function will most likely not be an issue after large swap boxes are not in use anymore. |
@NikolaiLong, the circuit is just an example, it has no particlar name. |
Debugging of failing testcase complete -
ETA: On Track for 3/7/2025 |
I've been much busier with work this week than anticipated, pushing eta back: Updated Completion ETA: 3/14/2025 |
All tests passing locally, pdf test is out of scope for this issue and has been left commented out. |
I apologize for the in-activity on this issue. I have had to study for the past month and will need continue for potentially a couple months. I will try to work in some time to continue development of this solution in my free time. |
Is your feature request related to a use case or problem? Please describe.
There are common notations for some gates supported by QCircuit, but Cirq's circuit_to_latex_using_qcircuit doesn't use them. This includes:
Describe the solution you'd like
Add special handling for common conventional notations (supported by QCircuit) to
circuit_to_latex_using_qcircuit
.[optional] Describe alternatives/workarounds you've considered
I wrote a find-and-replace post-processing script which produces desired result in my case:
[optional] Additional context (e.g. screenshots)
Example of circuit, its current rendering and desired rendering (which was obtained with a script above):
What is the urgency from your perspective for this issue? Is it blocking important work?
P3
The text was updated successfully, but these errors were encountered: