-
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
Added conj to qml math, fixed some ops #1143
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov Report
@@ Coverage Diff @@
## master #1143 +/- ##
=======================================
Coverage 97.97% 97.97%
=======================================
Files 142 142
Lines 10279 10289 +10
=======================================
+ Hits 10071 10081 +10
Misses 208 208
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 @Thenerdstation 💯
@@ -605,7 +606,7 @@ def _heisenberg_rep(p): | |||
|
|||
def adjoint(self, do_queue=False): | |||
U = self.parameters[0] | |||
return Interferometer(U.conj().T, wires=self.wires, do_queue=do_queue) | |||
return Interferometer(qml_math.T(qml_math.conj(U)), wires=self.wires, do_queue=do_queue) |
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.
🙏
Co-authored-by: Josh Izaac <josh146@gmail.com>
Context:
qml.math.conj
didn't exist before. This is needed to make adjoint transforms differentiable.Description of the Change:
Added
qml.math.conj
and fixed affected adjoint transforms.Benefits:
qml.math.conj
can now be used, adjoints are now differentiable transforms.Possible Drawbacks:
None
Related GitHub Issues:
None