-
Notifications
You must be signed in to change notification settings - Fork 603
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
Deprecate max expansion from qml.device
and qml.QNode
#6026
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 #6026 +/- ##
==========================================
- Coverage 99.66% 99.65% -0.01%
==========================================
Files 427 427
Lines 41059 40774 -285
==========================================
- Hits 40920 40634 -286
- Misses 139 140 +1 ☔ View full report in Codecov by Sentry. |
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.
One blocking comment left above for now. I'll give a 2nd pass after this is addressed.
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.
It looks all good to me!
Context:
max_expansion
only serves to prevent decompositions from reach the gateset they need to reach. Therefore, we are deprecating it in 0.38 and removing it in 0.39.Description of the Change:
The max_expansion is deprecated from
qml.QNode
and the analogousdecomp_depth
is deprecated fromqml.device
Benefits:
Specifying this only prevents the decomposition for reaching the target gateset, leading to things falling over uncontrollably later. Deprecating it (and later removing it) will avoid this.
Possible Drawbacks:
Users currently utilizing this argument must update their code after 0.39.
[sc-64693]