-
Notifications
You must be signed in to change notification settings - Fork 72
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
dialects: (stim) add qubit attribute and qubit coordinate attribute #3114
Conversation
…d files in one folder
Co-authored-by: Sasha Lopoukhine <superlopuh@gmail.com>
Co-authored-by: Sasha Lopoukhine <superlopuh@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3114 +/- ##
=======================================
Coverage 90.05% 90.06%
=======================================
Files 435 435
Lines 54647 54736 +89
Branches 8473 8490 +17
=======================================
+ Hits 49214 49299 +85
Misses 4060 4060
- Partials 1373 1377 +4 ☔ View full report in Codecov by Sentry. |
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.
This PR adds some code and infrastructure for stim printing, but never tests it. We should either add the printing code in a future PR or add the tests here.
Clean up some of the parser implementation ie. use in_angle_brackets Co-authored-by: Sasha Lopoukhine <superlopuh@gmail.com>
Let's merge this? |
…3114) Add a base attribute to the stim dialect - StimAttr which is StimPrintable so that all implementing attributes must implement print_stim. Add two initial attributes - QubitAttr to indicate qubits, and QubitCoordAttr which provides a pair of a coordinate of a physical qubit to a QubitAttr --------- Co-authored-by: Emilien Bauer <papychacal@gmail.com> Co-authored-by: Sasha Lopoukhine <superlopuh@gmail.com>
Add a base attribute to the stim dialect - StimAttr which is StimPrintable so that all implementing attributes must implement print_stim.
Add two initial attributes - QubitAttr to indicate qubits, and QubitCoordAttr which provides a pair of a coordinate of a physical qubit to a QubitAttr