-
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
Generalize Rotosolve to arbitray-frequency QNodes #2081
Conversation
Hello. You may have forgotten to update the changelog!
|
[sc-9758] |
Codecov Report
@@ Coverage Diff @@
## master #2081 +/- ##
=======================================
Coverage 99.18% 99.18%
=======================================
Files 226 226
Lines 17380 17388 +8
=======================================
+ Hits 17239 17247 +8
Misses 141 141
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 @dwierichs, this is a great PR to finish the work we started last year on Fourier analysis of variational circuits!
Very nice documentation and logic. I think this is very near merge-ready, the only reason I request changes is
- consider adding a default optimization method to reduce complexity for end-users
- so that we can be clear regarding breaking changes in the changelog 🙂
Co-authored-by: Josh Izaac <josh146@gmail.com>
Co-authored-by: Josh Izaac <josh146@gmail.com>
…to gen-rotosolve-2
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.
Great work @dwierichs 🙌
This implements the generalization of Rotosolve to arbitrary frequencies, further extending its use cases.
The last generalization #1489 allowed for arbitrary integer-valued frequencies, now any frequency spectra can be tackled.
RotosolveOptimizer
makes use ofqml.fourier.reconstruct
to obtain analytical reconstructions of univariate restrictions of the QNode of interest, and then minimizes this analytical reconstruction, using numerical methods in general and an analytic expression for the minimum for single-frequency functions.This implements #2058.