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
[Transform #2] Transforms program #4187
Conversation
[sc-36419] |
Hello. You may have forgotten to update the changelog!
|
Codecov Report
@@ Coverage Diff @@
## master #4187 +/- ##
========================================
Coverage 99.77% 99.77%
========================================
Files 347 352 +5
Lines 31452 31649 +197
========================================
+ Hits 31381 31578 +197
Misses 71 71
|
Co-authored-by: BorjaRequena <59647767+BorjaRequena@users.noreply.github.com>
Co-authored-by: BorjaRequena <59647767+BorjaRequena@users.noreply.github.com>
Co-authored-by: BorjaRequena <59647767+BorjaRequena@users.noreply.github.com>
Co-authored-by: BorjaRequena <59647767+BorjaRequena@users.noreply.github.com>
…I/pennylane into transforms_dispatcher
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.
Thanks for the changes Romain! I just have one question and after that I'm ready to approve.
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.
Thanks Romain! There's a test coverage issue, but that will need to be fixed before merging anyway so I'm happy to approve.
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.
Thanks for the great work, @rmoyard!
Description of the Change:
This PR introduces the main object of the transform rework, the Transform Program. We introduce it as a class with mulitple methods to interact with it.
Benefits:
We can store Transform programs on the QNode.