-
Notifications
You must be signed in to change notification settings - Fork 36
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
Stim DecodingGraph #356
Merged
quantumjim
merged 3 commits into
qiskit-community:339-integrate-stim
from
hetenyib:patch-2
Apr 5, 2023
Merged
Stim DecodingGraph #356
quantumjim
merged 3 commits into
qiskit-community:339-integrate-stim
from
hetenyib:patch-2
Apr 5, 2023
Conversation
This file contains 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
New methods: 1) 'stim_circuit_with_detectors' converts the qiskit circuit to stim and appends detectors 2) '_make_syndrome_graph' creates a rustworkx graph and hyper edge list to be passed onto the DecodingGraph class
New function: 'detector_error_model_to_rx_graph' creates a qiskit style decoding graph from a stim circuit
grace-harper
pushed a commit
that referenced
this pull request
Jun 9, 2023
* Code circuit builder -> integrate stim (#340) * create CodeCircuit class (#329) (#332) * create CodeCircuit class * add more detail to init * add default is_cluster_neutral * add css code circuit * Adding two methods: 'to_stim_circuit' and 'get_counts_via_stim' (#1) * Update css_code.py * Update css_code.py * Update css_code.py * stim integration in the CssCodeCircuit class (#341) * Update css_code.py * Update css_code.py * Update css_code.py * add stim * separate stim functions from class * move stim tools to utils * black and lint * document stim tools * document stim tools * add general noisification of circuits * add support for StabSubsystemCode * conflict avoidance * Update __init__.py * fix conflict * complete StabSubsystem compatibility * Stim DecodingGraph (#356) * decoding graph via stim New methods: 1) 'stim_circuit_with_detectors' converts the qiskit circuit to stim and appends detectors 2) '_make_syndrome_graph' creates a rustworkx graph and hyper edge list to be passed onto the DecodingGraph class * qiskit style decoding graph from stim circuit New function: 'detector_error_model_to_rx_graph' creates a qiskit style decoding graph from a stim circuit * Update decoding_graph_attributes.py * undo conflict fix * undo conflict resolution (#358) * Test (for CSS codes), black, lint, py39 tests (#361) * black + lint * Add files via upload * Update stim_tools.py * Update css_code.py * Grace's change requests (#363) 1) stim repeat block is still needed 2) specify what noise model means * per review * Documentation and functions (#374) * fix linting issues --------- Co-authored-by: hetenyib <[email protected]>
grace-harper
pushed a commit
that referenced
this pull request
Oct 2, 2023
* Code circuit builder -> integrate stim (#340) * create CodeCircuit class (#329) (#332) * create CodeCircuit class * add more detail to init * add default is_cluster_neutral * add css code circuit * Adding two methods: 'to_stim_circuit' and 'get_counts_via_stim' (#1) * Update css_code.py * Update css_code.py * Update css_code.py * stim integration in the CssCodeCircuit class (#341) * Update css_code.py * Update css_code.py * Update css_code.py * add stim * separate stim functions from class * move stim tools to utils * black and lint * document stim tools * document stim tools * add general noisification of circuits * add support for StabSubsystemCode * conflict avoidance * Update __init__.py * fix conflict * complete StabSubsystem compatibility * Stim DecodingGraph (#356) * decoding graph via stim New methods: 1) 'stim_circuit_with_detectors' converts the qiskit circuit to stim and appends detectors 2) '_make_syndrome_graph' creates a rustworkx graph and hyper edge list to be passed onto the DecodingGraph class * qiskit style decoding graph from stim circuit New function: 'detector_error_model_to_rx_graph' creates a qiskit style decoding graph from a stim circuit * Update decoding_graph_attributes.py * undo conflict fix * undo conflict resolution (#358) * Test (for CSS codes), black, lint, py39 tests (#361) * black + lint * Add files via upload * Update stim_tools.py * Update css_code.py * Grace's change requests (#363) 1) stim repeat block is still needed 2) specify what noise model means * per review * Documentation and functions (#374) * fix linting issues --------- Co-authored-by: hetenyib <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR contains two new methods in the CSSCodeCircuit class to generate a stim circuit with detectors; and a _make_syndrome_graph method that calls a util function from stim_tools generating a decoding graph. Additionally, the Node and Edge classes in decoding_graph_attributes.py needed a get attribute to enable pymatching to use the DecodingGraph.graph.