-
Notifications
You must be signed in to change notification settings - Fork 21
Pretty print #245
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
Open
benjvmin93
wants to merge
230
commits into
TeamGraphix:master
Choose a base branch
from
benjvmin93:pretty-print
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Pretty print #245
Changes from all commits
Commits
Show all changes
230 commits
Select commit
Hold shift + click to select a range
1699fe8
Refacto including new command classes. Some tests are still not passi…
benjvmin93 459287b
tests OK
benjvmin93 4fe2f97
new circuit instruction classes
benjvmin93 8e522a0
apply changes from instruction class in benchmark method
benjvmin93 f8bbcbd
made control instructions inherit from OneQubitInstruction
benjvmin93 9ee5d97
update instruction determination from name to class instance + remove…
benjvmin93 bfb2a53
update match for instructions and commands with an attribute to diffe…
benjvmin93 7829e73
improved _find_op_to_be_moved + test execution speed
benjvmin93 35ace68
improve _reorder_patter
benjvmin93 1e98cab
warnings to see execution time for tests
benjvmin93 766de39
black formatting
benjvmin93 83d979a
removed debug warnings + black
benjvmin93 ede1a78
fixed all conflicts
benjvmin93 eda4a46
typing + replace match by if/elif/else and make tests pass
benjvmin93 e840905
format
benjvmin93 923a15b
Merge branch 'master' into master
benjvmin93 d468480
fix test
benjvmin93 6e17dbf
Merge branch 'master' into master
benjvmin93 1bc3992
Merge remote-tracking branch 'upstream/master'
benjvmin93 d242e06
changes for PR
benjvmin93 e23caee
update plane attribute from M command to graphix.pauli.Plane
benjvmin93 037315a
update get_meas_plane more symbolic implem
benjvmin93 aacc569
more symbolic get_meas_plane using Pauli and Plane methods
benjvmin93 d11fd82
updates related to pr
benjvmin93 791378a
merged remote repo + update apply_correction method with command class
aba3ebb
fixed test + update apply_clifford method
b0c05ec
Fix diff
thierry-martinez c19d12b
Black transpiler
thierry-martinez a38b2a4
Fix review comments
thierry-martinez 6d9c897
Remove match
thierry-martinez 67b365d
Update get_corrections_from_pattern
thierry-martinez cd6501c
Add test for get_corrections_from_pattern
thierry-martinez 026c41c
Remove comments
thierry-martinez dc94a37
Remove comments
thierry-martinez 8fcd1b8
Remove comments
thierry-martinez 9789756
Remove comments
thierry-martinez 778f709
Check measure planes in gflow functions
thierry-martinez 474165f
:heavy_plus_sign: Add eval_type_backport
EarlMilktea 47242df
:recycle: Use __future__
EarlMilktea 1c22911
Merge pull request #1 from TeamGraphix/review-pr155
benjvmin93 6c0e690
first commit pretty_print + notebook example
benjvmin93 24e17dc
turn dict into list of str + format
benjvmin93 a93c662
added to_qasm3 method. Need to implement to_QASM3() for instructions
benjvmin93 b314626
added abstract method to_qasm3 instruction
benjvmin93 29f6b1f
added to_qasm3 method for circuit instructions
benjvmin93 e3d75d4
circuit and instructions to_qasm3 methods
2d3864e
requirement for qiskit visualiwation
26b8a91
update str + circuit.draw methods
da6ca35
added test to make sure circuit.draw() doesn't raise exception.
9b572d2
to_latex pattern + command methods
df5a733
Merge remote-tracking branch 'upstream/master'
a2a0f2e
Merge branch 'master' into pretty-print
9cb6c0f
fix instruction file to synchronize with master
fe18260
rewrote circuit pretty print for master sync
fe29cb7
remove old useless notebook
d41d1f0
moved qiskit import directly into draw() method + remove useless matp…
d389062
update print_pattern to __str__ + draw method for pattern. Need to fi…
76cced9
fix _latex_file_to_image method
8662b9e
remove debug print
39e44bd
__str__ method for commands
12eb951
restored print_pattern + update method name to to_png
e87c3d0
cmd to unicode
3e4365d
pattern to_unicode method
34d9e32
added qiskit_qasm3_import for circuit drawing
295781e
fixed small issues in command draw representations
5d6960f
factorized code with to_latex method that only returns a string of th…
2fb5a72
Merge remote-tracking branch 'upstream/master'
b9b3a47
sync master with pretty-print branch
b98cba7
Merge branch 'master' into pretty-print -> removed get_local_pattern …
49afeaa
added return type for drawing methods
3a02de4
update changelog for PR
65d5381
ruff formatting
b9d67d8
format
67994aa
consistency with master sync
3595c49
format
647e6df
added documentation + format
1b8a5f8
ruff
04cf092
added draw("png") in test_pattern
f3aa1cb
update draw tests
benjvmin93 62e01a4
ruff for PR
c0af200
ruff for PR
2f633c7
mypy PR
d55a6bc
typing
02ba115
ruff
45708d6
removed command draw methods from KindChecker class
8548b2b
update to_qasm3 arg type to Instruction
feb79d1
fix mypy errors
da31961
ruff format
cbf3851
changed deprecated typing
0885fd2
removed unused variables
87aa6d4
added setup texlive github action
058a919
moved call to texlive setup action in cov.yml
18b2a6d
fix error cov.yml
8a397ea
added poppler-utils
becf927
put texlive installation in CI.yml
6c7460e
texlive installation in cov.yml
2343b93
fix yaml syntax
d77bb43
Update cov.yml
benjvmin93 53c3530
added sudo for permissions
fdd679b
Merge branch 'pretty-print-integrate-texlive-ci' of github.com:benjvm…
23e4bf5
Update ci.yml
benjvmin93 067f3b3
Update cov.yml
benjvmin93 835c334
added -y option in CI
f078847
added qcircuit latex package
4a9262e
locate debug
d34648f
update CI
954a761
remove updatedb
e4fdc35
added qcircuit package in github texlive action
2ab157b
update latex packages to scheme-full
aa4535e
removed scheme-full for specific packages
ad6d4df
set back to scheme-basic
3ced4ad
write permission for generating pdf files from latex ?
057ee4f
added debug action
af01ff9
add package dependencies
9399e73
xypic
e764f55
remove tmate session action
5288b40
fix xypic name + cleaned ci
3959bf9
skip tests if no latex installed
ea708a8
update CI for specific os
ff391fe
ruff format fix
a955965
format
6a665ee
update poppler
be244f3
change find_executable to which bc of deprecation
9610edc
moved pylatexenc from requirements-dev to requirements
ea43b36
cleaned command file using f string convention, reduced redundant cod…
a2108f5
use f string convention in to_qasm3 Instruction method
ebca302
remove print debug
9e303ca
cleaner code: use pathlib + move builtin module imports at toplevel
e397339
ruff format
cbd61b8
format
ebd0e5a
typing in test
1a99ffb
skipping test ibmq backend
7130ca0
check for exception in str (ie. qiskit not available) and return repr…
75c8a76
fix bug in to_qasm3
8a0fc8a
added reverse_composition param to pattern drawing
6db71c2
format
8587ae9
rewrite command to str and unicode
7502821
rewrite to latex function
87ac331
pretty print new tests
7607a86
set default value for reverse_composition arg
d48d56f
fix tests
aa79fbb
fix command drawings
4798719
wrongly inverted s_domain and t_domain in drawing representations
c76bb38
added latex separator for to_latex + corrected tests writing
8d115cd
fix bug in to_qasm3 for rotation gates
bebd826
moved pylatex and qiskit_qasm dependencies to requirements-extra + ad…
323348d
format
5025bbb
rename reverse_composition argument to left_to_right
f5ea223
fix test to qasm3 import => tests are now running but assertions fail
4c66845
ruff format
a1cb2ec
lint
3e2d34a
removed graphix-perceval from extra dependencies as it doesn't work a…
80fa7cb
skip circuit draw test is qiskit is not installed as it depends on it
7b21abe
fix forgotten sys impot to allow check if qiskit module is installed
3acfdd4
format
af41507
skip for optional dependencies in transpiler test
f96e24a
update instruction to qasm for consistency with pyzx + added related …
b910f99
lint + format
bc86db5
put subscript and superscript in const
benjvmin93 655a066
cleaned command pretty print + update tests
benjvmin93 fa8c41a
changes in pattern.__repr__() method
6bfa593
repr for commands
4e1b000
commented test_to_qasm3 as pyzx is broken
82ccbe3
format
0749e6d
lint
51127b4
remove T from to_latex
51f484f
Merge remote-tracking branch 'upstream/master'
ffeefce
Merge branch 'master' into pretty-print
21adfcf
removed T from pretty print methods
28dc783
fix T.__repr__
b626d99
lint
7ce1c9e
skip test if pyzx not installed
c030f41
FIX typo
8894a0a
removed graphix ibmq as pip Could not find a version that satisfies t…
eb63332
try to check windows error latex
2efa42d
fix unnecessary mode
681c1b6
removed debug print => Windows ci is not passing due to differences i…
8c5dd7a
put qiskit.qasm3.loads at file top level since qiskit is in requireme…
1685543
Merge branch 'master' into pretty-print
benjvmin93 808cd30
added extra for test + ruff format
d7dd286
Merge branch 'pretty-print' of github.com:benjvmin93/graphix into pre…
238ca28
format
5b05ad4
fix typo for ci
b01a51d
moved import qiskit with try except since we only use it inside draw
d192341
skip if pyzx is missing
49d2502
removed extra deps for mypy checks
3bbf0ea
issues with poppler version with choco => downloading binary from mai…
6c79fd2
fix windows path in CI
b3a82f1
fix bad windows path in CI
426ca05
reset windows ci + skip if pdftocairo not available despite poppler i…
af935a8
install poppler dependencies for windows without choco
aafc366
trying to fix paths for windows ci poppler
988e9a9
use curl instead of gitclone
1855c09
trying to fix path
a5f39c4
trying to fix CI
102115a
trying to update the PATH right
9a7e4f4
use expand-archive instead of tar
3ce71e6
using Invoke-RestMethod instead of downloading curl
4ccf32a
debug
521e761
Bump ruff from 0.9.10 to 0.11.0 in the python-packages group
dependabot[bot] eb31b32
fix angle type to ExpressionOrFloat
benjvmin93 0f0d7da
ruff format
benjvmin93 f312c69
skip test if on windows
benjvmin93 bdd8de8
skip tests requiring pdftocairo if os is windows
benjvmin93 5859646
fixed ruff errors using dict.fromkeys method
f9c4de1
Merge branch 'dependabot/pip/python-packages-002828dd18' into pretty-…
a238143
update repr methods for commands
b84be0e
improve test for multiple nodes in domains
1226995
update empty set check to checking length
4bc4123
add PTH rule for ruff linter + use pathlib.open
3a55f5e
use reversed function instead of slicing
2bc73db
update _angle_to_str method for command drawing
benjvmin93 29ad533
fix typos for fraction in latex representation
benjvmin93 c2415d2
Merge branch 'master' into pretty-print
benjvmin93 1cdb082
New argument seq for Pattern constructor, and change repr
benjvmin93 8314903
Merge branch 'pretty-print' of github.com:benjvmin93/graphix into pre…
benjvmin93 1708489
fix linter error
benjvmin93 045bcf4
commented poppler-windows setup in CI
e379ed2
removed private method mentions
c32886a
Merge branch 'pretty-print' of github.com:benjvmin93/graphix into pre…
db1ed3c
suggested changes for reprensenting domain inside S command
benjvmin93 04850a9
changes in _angle_to_str: change parameters with mode for properly re…
07d5709
Merge branch 'pretty-print' of github.com:benjvmin93/graphix into pre…
22ef798
fix angle_to_str + test
f68356c
added pattern.py to mypy checks
af091ab
remove pattern and add newly created draw_pattern file for mypy check
8fe8469
ruff lint + format
a16f67a
update test drawing pattern + fix inside command for drawing clifford
dfbe04f
update instruction to_qasm3 function based on Thierry's suggestion
543dc16
ruff format + linter
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that if we get rid of the Windows test for TeX, this change is not needed.