-
Notifications
You must be signed in to change notification settings - Fork 575
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
Commutation dag #1712
Commutation dag #1712
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov Report
@@ Coverage Diff @@
## master #1712 +/- ##
==========================================
+ Coverage 99.22% 99.27% +0.05%
==========================================
Files 231 232 +1
Lines 18392 18708 +316
==========================================
+ Hits 18249 18573 +324
+ Misses 143 135 -8
Continue to review full report at Codecov.
|
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
…ylane into commutation_dag
[sc-15667] |
Context:
This PR adds a
is_commuting
checks if two operations commute, it does not use matrices multiplications but a truth table. It also uses some matrix multiplication for edge cases.Benefits:
A commutation function that does not use matrices multiplication and takes any two operations as inputs. A new DAG that represents the commutation structure of quantum circuits and is useful for quantum circuits optimization.