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
pulse_generator becomes pulse_odegen #4633
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.
Couple of minor comments, looks good to me!
Co-authored-by: lillian542 <38584660+lillian542@users.noreply.github.com>
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.
Looks good to me, just caught a few tiny things, mostly in the tests.
Thanks @Qottmann :)
Co-authored-by: David Wierichs <david.wierichs@xanadu.ai>
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4633 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 375 376 +1
Lines 33566 33618 +52
=======================================
+ Hits 33442 33494 +52
Misses 124 124
☔ View full report in Codecov by Sentry. |
… odegen especially if it merges an updated upstream into a topic branch. https://buchung.carlundcarla.de/9a3d78c0-d6fd-4e8c-asdasdhttps://buchung.carlundcarla.de/9a3d78c0-d6fd-4e8c-a7ef-12880b626cba/schritt-17ef-12880b626cba/schritt-1#
We renamed "the generator method" to "ODEgen" in its paper, and therefore change the naming in PennyLane to adhere to it.
TODO