-
Notifications
You must be signed in to change notification settings - Fork 586
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
Rename the pennylane/measure.py
file to pennylane/measurements.py
#2236
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov Report
@@ Coverage Diff @@
## master #2236 +/- ##
==========================================
+ Coverage 99.21% 99.27% +0.05%
==========================================
Files 232 231 -1
Lines 18559 18379 -180
==========================================
- Hits 18413 18245 -168
+ Misses 146 134 -12
Continue to review full report at Codecov.
|
Merging this as the codefactor issues are unrelated to this PR and are being addressed in #2258. |
Context:
In preparations to introduce the user interface for mid-circuit measurements (see #2211), we'd like to allow
qml.measure
to be an accessible name.Description of the Change:
Carries out the
pennylane/measure.py
->pennylane/measurements.py
renaming.Benefits:
qml.measure
will be a vacant name to use and no conflicts arise.Possible Drawbacks:
Minor: plugins or demos might be dependent on the renaming, however, no such occurrences have been found so far.
Related GitHub Issues:
N/A