-
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
Elimate requires_grad=True
warnings from device tests
#1913
Conversation
[sc-11205] |
Codecov Report
@@ Coverage Diff @@
## master #1913 +/- ##
=======================================
Coverage 98.83% 98.83%
=======================================
Files 222 222
Lines 16979 16979
=======================================
Hits 16781 16781
Misses 198 198
Continue to review full report at Codecov.
|
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 this :)
Just needs a changelog entry and I'll 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.
:) Just the one spelling thing.
Small PR to remove
requires_grad=True
warnings from the device tests in:These warnings were introduced in anticipation of Pennylane v0.20.0 making the default
requires_grad=False
for variational parameters.