-
Notifications
You must be signed in to change notification settings - Fork 616
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 test_simplify_with_adjoint_not_defined
in test_pow_ops.py
#6388
Conversation
Hello. You may have forgotten to update the changelog!
|
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.
Don't think this needs a changelog entry, but I'm fine leaving it in anyway.
Thanks for catching this.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6388 +/- ##
=======================================
Coverage 99.39% 99.39%
=======================================
Files 448 448
Lines 42464 42466 +2
=======================================
+ Hits 42209 42211 +2
Misses 255 255 ☔ View full report in Codecov by Sentry. |
…6388) This PR renames `test_simplify_with_adjoint_not_defined` to `test_simplify_with_pow_not_defined` in the `test_pow_ops.py` file. The test is given by the following code. ``` def test_simplify_with_adjoint_not_defined(self): """Test the simplify method with an operator that has not defined the op.pow method.""" op = Pow(qml.U2(1, 1, 0), z=3) simplified_op = op.simplify() assert isinstance(simplified_op, Pow) assert op.data == simplified_op.data assert op.wires == simplified_op.wires assert op.arithmetic_depth == simplified_op.arithmetic_depth ``` Note the function name says `adjoint` is not defined, but the docstring says that `pow` is not defined. The docstring is correct, since `qml.U2` defines the `adjoint` method.
…6388) This PR renames `test_simplify_with_adjoint_not_defined` to `test_simplify_with_pow_not_defined` in the `test_pow_ops.py` file. The test is given by the following code. ``` def test_simplify_with_adjoint_not_defined(self): """Test the simplify method with an operator that has not defined the op.pow method.""" op = Pow(qml.U2(1, 1, 0), z=3) simplified_op = op.simplify() assert isinstance(simplified_op, Pow) assert op.data == simplified_op.data assert op.wires == simplified_op.wires assert op.arithmetic_depth == simplified_op.arithmetic_depth ``` Note the function name says `adjoint` is not defined, but the docstring says that `pow` is not defined. The docstring is correct, since `qml.U2` defines the `adjoint` method.
This PR renames
test_simplify_with_adjoint_not_defined
totest_simplify_with_pow_not_defined
in thetest_pow_ops.py
file. The test is given by the following code.Note the function name says
adjoint
is not defined, but the docstring says thatpow
is not defined. The docstring is correct, sinceqml.U2
defines theadjoint
method.