-
Notifications
You must be signed in to change notification settings - Fork 368
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
Improve gate fusion #667
Improve gate fusion #667
Conversation
67e5ed6
to
3c2904b
Compare
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.
Looks good! I have only a few comments/suggestions.
76f7ac0
to
e1f60c0
Compare
e1f60c0
to
6d4278c
Compare
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've pointed out some format/typos and I suggest change a copy operation with an inserter to a set insert operation. Consider these suggestions, but other than that everything looks good and ready to merge.
6d4278c
to
15f67d3
Compare
This allows fusion of arbitrary number of qubits and any operation supported by the unitary simulator
Currently this only applies for density matrix simulation method Add more detailed fusion metadata
This can now be gotten through input_ops from verbose
19e24fe
to
b1995c0
Compare
Summary
Refactor gate fusion transpilation to use the Unitary state simulator or superoperator state simulator for fusing instructions. This also allows fusion of superoperator instructions in noisy circuits for density matrix simulation.
Details and comments
Partial closes #616