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

added draw method to circuit #266

Merged
merged 3 commits into from
Jan 25, 2023

Conversation

JamesB-1qbit
Copy link
Contributor

Uses cirq's SVGCircuit such that the following code

from tangelo.linq import Circuit, Gate

circ = Circuit([Gate("CRZ", target=0, control=[1,2], parameter=1),
                Gate("CNOT", target=0, control=1),
                Gate("RZ", target=0, parameter=0.3)])
circ.draw()

makes
output.svg.pdf

@ValentinS4t1qbit
Copy link
Collaborator

ValentinS4t1qbit commented Jan 18, 2023

Thank you for this simple solution, James. Since Cirq is a requirement of Openfermion, itself a requirement of our package, I think this is a very reasonable default solution.

When I try to call the draw method in the basics notebook, I get a bunch of warning for a font not found, in my Ubuntu environment. I think 1 warning line per text box, which will quickly get unbearable. Is there any way we can silence those warnings or not having them come up in the first place ?

Screenshot from 2023-01-17 17-43-39

@JamesB-1qbit
Copy link
Contributor Author

I don't see these warnings and my virtual box seems to be broken. But I think that

warnings.filterwarnings("ignore", message=".*findfont.*")

I will commit this change and then someone who uses linux can test it. This error shows up on a bunch of cirq notebooks.

Copy link
Collaborator

@ValentinS4t1qbit ValentinS4t1qbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you !

@ValentinS4t1qbit ValentinS4t1qbit merged commit 3cc81c8 into sandbox-quantum:develop Jan 25, 2023
ValentinS4t1qbit added a commit that referenced this pull request Feb 9, 2023
* Update develop to catch up with main version 0.3.3 (#249)

* Fix link of fig in qchem & excited states notebook (#250)

* allow single flip index dis for qcc (#247)

* allow single flip index dis for jkmn

* Richardson extrapolation: bug fix + error estimation (#252)

* Bugfix: DMET with QCC (#253)

* iQCC using only Clifford circuits notebook (#254)

* pUCCD ansatz (#251)

* UHF reference (#240)

* uhf implementation with VQESolver functionality
* support for all types of orbital freezing
* add active_spin and uhf attributes to SecondQuantizedDMETFragment
* add spin to adapt_ansatz arguments

* added multi-product, grid_circuits and discrete_clock (#257)

* added multi-product, grid_circuits and discrete_clock

* translation to pennylane (#260)

* tangelo to pennylane format translation

Co-authored-by: Valentin Senicourt <[email protected]>

* bump testing version to 3.8 (#262)

Updating python version to 3.8 in automated tests, as 3.7 is no longer maintained by the Python dev team

* Auto threshold cutoff for small coefficients in LCU (#261)

* check for small value lcu
* changed to keep same vector length but apply no operations

* Openshell DMET (#208)

* Open-shell DMET.
* Fix for get_rdm CCSD.
* Added NAO localization.
* Added LiO2 spin=1 DMET test.
* Added UHF MF for DMET. New get_rdm for VQESolver.

* Save mid-circuit measurement (#256)

* A flag now allows users to save mid-circuit measurements for each shot run.

Co-authored-by: Valentin Senicourt <[email protected]>

* Fix for IBMQConnection (#264)

* Updated copyright year to 2023 (#267)

* added draw method to circuit (#266)

* added draw method to circuit (warning about font comes up in linux env)

* Cirq, qulacs, pennylane and projectq operator translation functions (#268)

* Support to bidirectional translation of operators for cirq, qulacs, PennyLane, projectq.
* Fixed a bug in QulacsSimulator.expectation_value_from_prepared_state.
* Important bug in qiskit -> tangelo op translation fixed

* Fixed identity operator evaluation and extended the accepted input types in compute_rdms (#269)

* Compute RDMs function can now hangle both term formats (string and openfermion tuple).
* Unhashable list for of -> conversion to tuples.
* Handling all the inputs in compute_rdms.

* Temporarily disabling OS-DMET (#271)

* Temporarily disabling for code verification purpose, as the implementation seems to return very odd and incorrect results for simple systems.

* Link to new example repo

* Update README.rst

* Revert changes that broke tests (#275)

* reverted changes that broke tests

---------

Co-authored-by: AlexandreF-1qbit <[email protected]>
Co-authored-by: James Brown <[email protected]>
Co-authored-by: KrzysztofB-1qbit <[email protected]>
ValentinS4t1qbit added a commit that referenced this pull request Feb 16, 2023
* Update develop to catch up with main version 0.3.3 (#249)

* Fix link of fig in qchem & excited states notebook (#250)

* allow single flip index dis for qcc (#247)

* allow single flip index dis for jkmn

* Richardson extrapolation: bug fix + error estimation (#252)

* Bugfix: DMET with QCC (#253)

* iQCC using only Clifford circuits notebook (#254)

* pUCCD ansatz (#251)

* UHF reference (#240)

* uhf implementation with VQESolver functionality
* support for all types of orbital freezing
* add active_spin and uhf attributes to SecondQuantizedDMETFragment
* add spin to adapt_ansatz arguments

* added multi-product, grid_circuits and discrete_clock (#257)

* added multi-product, grid_circuits and discrete_clock

* translation to pennylane (#260)

* tangelo to pennylane format translation

Co-authored-by: Valentin Senicourt <[email protected]>

* bump testing version to 3.8 (#262)

Updating python version to 3.8 in automated tests, as 3.7 is no longer maintained by the Python dev team

* Auto threshold cutoff for small coefficients in LCU (#261)

* check for small value lcu
* changed to keep same vector length but apply no operations

* Openshell DMET (#208)

* Open-shell DMET.
* Fix for get_rdm CCSD.
* Added NAO localization.
* Added LiO2 spin=1 DMET test.
* Added UHF MF for DMET. New get_rdm for VQESolver.

* Save mid-circuit measurement (#256)

* A flag now allows users to save mid-circuit measurements for each shot run.

Co-authored-by: Valentin Senicourt <[email protected]>

* Fix for IBMQConnection (#264)

* Updated copyright year to 2023 (#267)

* added draw method to circuit (#266)

* added draw method to circuit (warning about font comes up in linux env)

* Cirq, qulacs, pennylane and projectq operator translation functions (#268)

* Support to bidirectional translation of operators for cirq, qulacs, PennyLane, projectq.
* Fixed a bug in QulacsSimulator.expectation_value_from_prepared_state.
* Important bug in qiskit -> tangelo op translation fixed

* Fixed identity operator evaluation and extended the accepted input types in compute_rdms (#269)

* Compute RDMs function can now hangle both term formats (string and openfermion tuple).
* Unhashable list for of -> conversion to tuples.
* Handling all the inputs in compute_rdms.

* Temporarily disabling OS-DMET (#271)

* Temporarily disabling for code verification purpose, as the implementation seems to return very odd and incorrect results for simple systems.

* Link to new example repo

* Update README.rst

* Revert changes that broke tests (#275)

* reverted changes that broke tests

* DMET + frozen orbitals for each fragment (#276)

* DMET + frozen orbitals,  added DMET/frozen tests.

* Example folder relocated to new repo Tangelo-Examples (#274)

* removed examples folder, removed notebooks from automated tests, added link higher in README.rst

* Docs update (#272)

* Updated conf.py, tested building docs and fixed comments/docs.
* Removed tutorials.rst with the notebooks.

* Desired measurement result (#263)

* New feature: return frequencies matching desired mid-circuit measurement values.

---------

Co-authored-by: James Brown <[email protected]>

* Bumping Tangelo version number in _version.py

* Update Changelog (v0.3.4)

---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: AlexandreF-1qbit <[email protected]>
Co-authored-by: James Brown <[email protected]>
Co-authored-by: KrzysztofB-1qbit <[email protected]>
Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: James Brown <[email protected]>
Co-authored-by: GitHub Actions <[email protected]>
ValentinS4t1qbit added a commit that referenced this pull request Jun 29, 2023
* Update develop to catch up with main version 0.3.3 (#249)

* Fix link of fig in qchem & excited states notebook (#250)

* allow single flip index dis for qcc (#247)

* allow single flip index dis for jkmn

* Richardson extrapolation: bug fix + error estimation (#252)

* Bugfix: DMET with QCC (#253)

* iQCC using only Clifford circuits notebook (#254)

* pUCCD ansatz (#251)

* UHF reference (#240)

* uhf implementation with VQESolver functionality
* support for all types of orbital freezing
* add active_spin and uhf attributes to SecondQuantizedDMETFragment
* add spin to adapt_ansatz arguments

* added multi-product, grid_circuits and discrete_clock (#257)

* added multi-product, grid_circuits and discrete_clock

* translation to pennylane (#260)

* tangelo to pennylane format translation

Co-authored-by: Valentin Senicourt <[email protected]>

* bump testing version to 3.8 (#262)

Updating python version to 3.8 in automated tests, as 3.7 is no longer maintained by the Python dev team

* Auto threshold cutoff for small coefficients in LCU (#261)

* check for small value lcu
* changed to keep same vector length but apply no operations

* Openshell DMET (#208)

* Open-shell DMET.
* Fix for get_rdm CCSD.
* Added NAO localization.
* Added LiO2 spin=1 DMET test.
* Added UHF MF for DMET. New get_rdm for VQESolver.

* Save mid-circuit measurement (#256)

* A flag now allows users to save mid-circuit measurements for each shot run.

Co-authored-by: Valentin Senicourt <[email protected]>

* Fix for IBMQConnection (#264)

* Updated copyright year to 2023 (#267)

* added draw method to circuit (#266)

* added draw method to circuit (warning about font comes up in linux env)

* Cirq, qulacs, pennylane and projectq operator translation functions (#268)

* Support to bidirectional translation of operators for cirq, qulacs, PennyLane, projectq.
* Fixed a bug in QulacsSimulator.expectation_value_from_prepared_state.
* Important bug in qiskit -> tangelo op translation fixed

* Fixed identity operator evaluation and extended the accepted input types in compute_rdms (#269)

* Compute RDMs function can now hangle both term formats (string and openfermion tuple).
* Unhashable list for of -> conversion to tuples.
* Handling all the inputs in compute_rdms.

* Temporarily disabling OS-DMET (#271)

* Temporarily disabling for code verification purpose, as the implementation seems to return very odd and incorrect results for simple systems.

* Link to new example repo

* Update README.rst

* Revert changes that broke tests (#275)

* reverted changes that broke tests

* DMET + frozen orbitals for each fragment (#276)

* DMET + frozen orbitals,  added DMET/frozen tests.

* Example folder relocated to new repo Tangelo-Examples (#274)

* removed examples folder, removed notebooks from automated tests, added link higher in README.rst

* Docs update (#272)

* Updated conf.py, tested building docs and fixed comments/docs.
* Removed tutorials.rst with the notebooks.

* Desired measurement result (#263)

* New feature: return frequencies matching desired mid-circuit measurement values.

---------

Co-authored-by: James Brown <[email protected]>

* fix error for imaginary qubit_op with desired meas result (#278)

* Integrate bug fix into main before release (#278) (#279) (#280)

Co-authored-by: James Brown <[email protected]>
* fix error for imaginary qubit_op with desired meas result (#278)

* Depth function performance increase (#285)

* Depth function takes now less than half a second to compute it from scratch for ~120K depth circuit.

* Fix for (inverse of S and T gate) #287. (#288)

* Translation perf tests (operator, circuit) (#289)

* Performance test for translation layer of linq, for most formats

---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>

* deterministic desired_meas_result (#290)

* deterministic desired_meas_result

* Mifnohelper print: remove problem handle (#293)

* Removed problem handle from print. It now only focuses on chemistry information

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>

* Combinatorial mapping of fermionic Hamiltonians (#286)

* Initial implementation. Known performance scaling issues, may attempt to introduce a different QubitOperator in the future.

* Re-enabling open-shell DMET (#291)

* Re-enabling OS-DMET.

* Support for symbolic simulator (simpy) (#292)

Support for symbolic backend based on sympy, to provide algebraic expressions for states / circuits with less than 10 qubits. Notebook / examples to follow later on.

* Arg simulate_options and projective_circuit to VQESolver (#298)

* It is now possible to request desired measurements and other options for the "simulate" method called by the backend
* Projective circuits can be used with VQE algorithm(s). Either passed by the user explicitly or integrated through the ansatz.

---------

Co-authored-by: Valentin Senicourt <[email protected]>

* Supporting symbols as parameters and add nsimplify to results (#300)

* Symbolic expression simplified.

* Simplification of exp. value and fixed a bug in translate op.

* Add ability to get inverse with Symbol parameters.

* Switch nsimplify to evalf.

* Trim trivial qubits from circuit and Hamiltonian (#302)

Remove qubits that are in a deterministic state in quantum circuits, and simplify qubit Hamiltonian accordingly, in order to reduce resource requirements while computing expectation values.

---------

Co-authored-by: Valentin Senicourt <[email protected]>

* Push contributors towards develop and not to main for PRs (#303)

* Update to CONTRIB (main -> develop) and added workflow to help avoid PR to main that are not from develop

* Set the DMET virtual orbital truncation threshold at the user level (#304)

* Add option to turn off virt. orb. truncation.
* Changed the threshold instead of a flag. Occ. set to their abs values.

* Trim bug (#305)

* minor bug fixes + test

* Added typing for option dictionaries in quantum agorithms (#307)

* added typing for option dictionaries, which improves user experience (autocomplete, auto-loading of docstrings for different GUIs) and handle options better overall

* IntegralSolver class: base + support for Psi4 (#297)

* Tests for pyscf, test for psi4, tests for custom IntegralSolver
* temporary: install NO_PYSCF available to users
* new: MP2Solver

This PR will be followed by 1-2 other PRs to decouple pyscf from almost all algorithms and streamline the use of classical chem backends (support for all classical solvers, turning pyscf as an optional dependency for Tangelo, fixing docs and README etc as a result)

* Change multi-controlled CNOT to multi-controlled CX (#308)

* change multi-controlled CNOT to multi-controlled CX

---------

Co-authored-by: Valentin Senicourt <[email protected]>

* Braket connection (#312)

Support for Braket connection, including batch submission of jobs. Tests, docs, refresh of env_var.sh
A PR for the QPU notebook will follow

---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>

* Support for UMP2 initial parameters (#310)

* Support for UMP2 initial parameters.

* Checkfile for IntegralSolverPySCF (#311)

* chkfile for IntegralSolverPySCF.

* QubitOperator import from Tangelo, for better encapsulation (#299)

* Use Tangelo's `QubitOperator` instead of OpenFermion when possible.
* Constructor method `from_openfermion` and export method `to_openfermion` available in `QubitOperator` class.

Remaining usage of Openfermion is tied to qubit mappings mostly, and code relying on other Openfermion features.

---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>

* FCISolverPsi4 (#309)

* moved chem tests to molecular_computation tests
* Changed FCISolver to class with solver attribute

---------

Co-authored-by: Valentin Senicourt <[email protected]>

* Removed deprecated Linq functions (#316)

* Removed deprecated versions of trasnlation functions and Simulator.
---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>

* CCSD solver psi4 (#313)

* add ccsd_solver test to psi4 testing
* get_rdm currently not supported, suggestions added in comments

* ADAPT: add spin to available options  (#317)

* add spin to available options and docstring

* MP2 psi4 (#315)

* Support for MP2 with psi4 backend.

* get_rdm and get_mp2_parameters currently unsupported, suggestions left in docstrings.

* skip performance tests (Linq) in pytest (#319)

Perrformance tests take time and should not be run systematically

* Typos in docs (#321)

* Added spaces in messages.

* Typos and code format.

* Add period.

* DMET Effective Core Potential fix(related to #306) (#318)

* potential fix for #306, test added

* Change installation (#320)

* remove no_pyscf, attempt windows test
* updated psi4 test

---------

Co-authored-by: Valentin Senicourt <[email protected]>

* DMET-ecp test fix: change optimizer to minimize square (#323)

* improved Error message for MP2 initialization in UCCSD (pySCf currently required) (#322)

* improved Error message for MP2 initialization if pyscf is not found

* Qiskit-related: deprecation update to SparsePauliOp and bug workaround (#325)

* Stim clifford simulator integration (#314)

* direct tableau
* noise and circuit sampler
* decomposition into clifford gates
* clifford decomp tests, and SDAG in cirq
* is_clifford in gate class
* expand clifford decomp to integer values
* small comments, add pip install stim to workflow

Co-authored-by: Valentin Senicourt <[email protected]>

* Bumping Tangelo version number in _version.py

* Update CHANGELOG.md

---------

Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: AlexandreF-1qbit <[email protected]>
Co-authored-by: James Brown <[email protected]>
Co-authored-by: KrzysztofB-1qbit <[email protected]>
Co-authored-by: Valentin Senicourt <[email protected]>
Co-authored-by: James Brown <[email protected]>
Co-authored-by: ValentinS4t1qbit <[email protected]>
Co-authored-by: elloyd-1qbit <[email protected]>
Co-authored-by: GitHub Actions <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants