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
Update quantum monte carlo #4708
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4708 +/- ##
==========================================
- Coverage 99.49% 99.49% -0.01%
==========================================
Files 388 390 +2
Lines 35513 35265 -248
==========================================
- Hits 35335 35088 -247
+ Misses 178 177 -1 ☔ View full report in Codecov by Sentry. |
…ennylane into update_monte_carlo
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.
left some non-blocking style concerns, but lgtm!
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 left 2 comments, otherwise ready to approve :)
**Description of the Change:** - Update the quantum monte carlo to the new transform system - Upadte applied_controlle_Q to the new transform system - Update the documentation **Benefits:** The transform can now be applied directly on QNodes.
Description of the Change:
Benefits:
The transform can now be applied directly on QNodes.