pennylane
pennylane copied to clipboard
[Return-types #5.3] Param-shift shot vector case
Context:
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:
Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md
with:
- A one-to-two sentence description of the change. You may include a small working example for new features.
- A link back to this PR.
- Your name (or GitHub username) in the contributors section.
Codecov Report
Merging #3099 (acebd44) into master (351c6bd) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #3099 +/- ##
=======================================
Coverage 99.69% 99.69%
=======================================
Files 275 275
Lines 23944 24011 +67
=======================================
+ Hits 23870 23937 +67
Misses 74 74
Impacted Files | Coverage Δ | |
---|---|---|
pennylane/gradients/finite_difference.py | 100.00% <100.00%> (ø) |
|
pennylane/gradients/parameter_shift.py | 100.00% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
my above comments are all pretty small, this PR looks great! updating that test file was a true feat 😸
@timmysilv @rmoyard thanks for the great comments! :tada:
I think I've addressed them all, some with questions - keen to get further thoughts. :+1: Let me know what should definitely change still in this PR.