Skip to content

Graphstate Refactor with Descriptor #39

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.

Sign up for GitHub

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

Jump to bottom
Draft
masa10-f wants to merge 67 commits into
base: master
Choose a base branch
from
Draft

Graphstate Refactor with Descriptor #39

masa10-f wants to merge 67 commits into from

Conversation

@masa10-f
Copy link
Collaborator

@masa10-f masa10-f commented Apr 22, 2025

Try to simplify the methods of GraphState, though it's not an emergency.

masa10-f and others added 30 commits April 7, 2025 17:15
@masa10-f
implement graphstate developed in mf branch
7560209
@masa10-f
minimize API in BaseGraphState
b7641a3
@masa10-f
add backticks for intersphinx
1581cf7
@masa10-f
configure docs for graphstate
d9efcd4
@masa10-f
refactor q_index attribute
841dd35
@masa10-f
fix test of graphstate
f2a9a93
@masa10-f
specify error type
907ed1c
@masa10-f
modify interface of add_physical_node
ad42976
@masa10-f
update parse_input_local_cliffords
4ba895c
@masa10-f
remove append method
1f17cf5
@masa10-f
add random flow graph generator
ad5bd4e
@masa10-f
fix node removal
ebe0c1e
@masa10-f
update test for graphstate
4a653a3
@masa10-f
fix mypy error in numpy.number
000180b
@masa10-f
implement sequential composition
ab97a76
@masa10-f
update docstring
b29682f
@masa10-f
change the interface of q_index in input and output nodes
9c0106d
@masa10-f
update random object generator
46b8748
@masa10-f
update graphstate test
9216b15
@masa10-f
resolve TODO with sorted iterator
af77484
@masa10-f
implement parallel composition
426bfa0
@masa10-f
update doc
cd2ec92
@masa10-f
fix docstring
d4dc700
@masa10-f
update docs of graphstate
9f80ded
@masa10-f
fix bipartite_edges
c228ff9
@EarlMilktea
♻️ Iterate container directly
6b3b735
@EarlMilktea
♻️ Remove unnecessary TypeVar
acc536e
@EarlMilktea
♻️ Remove unnecessary raise
2c34853 
Because abstractmethod cannot be called in the first place
@EarlMilktea
♻️ Use unpack
044324b
@EarlMilktea
♻️ Use override deco
5c5e9ec
masa10-f added 27 commits April 20, 2025 19:10
@masa10-f
rename set_meas_basis -> assign_meas_basis
9f1e971
@masa10-f
rename funcs
34eded2
@masa10-f
update test and docs
475448b
@masa10-f
prohibit self-loop
b784929
@masa10-f
check meas basis is properly set
7089a02
@masa10-f
remove lc from composition
52af48c
@masa10-f
revert to use TypeVar
58338d2
@masa10-f
add comment
23cb79b
@masa10-f
add canonical form checker
55ae692
@masa10-f
canonical form check before composition
b91c023
@masa10-f
delete unnecessary branch
703867f
@masa10-f
fix docstring
d387750
@masa10-f
make pop_local_clifford private
41399af
@masa10-f
add output lc expansion method
31e3832
@masa10-f
update docstring
c2e967e
@masa10-f
add lc parser for composition
afad91f
@masa10-f
remove local clifford api from base class
ec0bbff
@masa10-f
use update_lc_lc in apply_local_clifford
4406a71
@masa10-f
implement canonical form checker
fc544f0
@masa10-f
check canonical form before composition
256b1df
@masa10-f
fix docstring
8a982e1
@masa10-f
remove redundunt properties
ed24a66
@masa10-f
fix docstring
8527b43
@masa10-f
add error cases in register input/output
7af1856
@masa10-f
update remove_output_node
b14f70b
@masa10-f
WIP: refactor meas_bases
4e6d54d
@masa10-f
update test
65662f5
@masa10-f
Copy link
Collaborator Author

masa10-f commented Apr 22, 2025

I left this branch to improve the API of the graphstate module. Nevertheless, it's not a priority because we currently plan to use this package as one of the backends of the FT-MBQC compiler. Therefore, the user-friendliness is not as important as I initially assumed.
(c.c. @EarlMilktea )

@EarlMilktea EarlMilktea changed the title Graphstate Refactor with Descripter Graphstate Refactor with Descriptor Apr 22, 2025
@masa10-f masa10-f self-assigned this Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@masa10-f masa10-f

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@masa10-f @EarlMilktea