-
Notifications
You must be signed in to change notification settings - Fork 102
Use HyperHessians instead of ForwardDiff to compute the hessian in the Landau example #1253
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1253 +/- ##
=======================================
Coverage 94.19% 94.19%
=======================================
Files 40 40
Lines 6662 6662
=======================================
Hits 6275 6275
Misses 387 387 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
termi-official
left a comment
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.
After some discussion on slack I think I am convinced for now that the typical user won't find a way to trigger perturbation confusion when sticking to this example.
|
Maybe I'll register the package and see if some people have an interest in poking on it before mergin thits. I would also like to get a reverse AD backend working on this to see what effect it has on performance. I started a DifferentiationInterface rewrite (17edffa#diff-bd098a9030f95d6512cf21c19a06a92acd264a6ca5b80f683ca379a0832abec5) but I got into troubles with every reverse AD backend I tried... |
|
Let me know if I can help with DI integration! |
|
The DI integration works fine and was very easy (great job) . It's just that no backend work... 😅 |
|
If you have Mooncake issues, don't hesitate to open them on the repo! It's a small team but they do their best! |
|
Ok, opened chalk-lab/Mooncake.jl#867. |
|
Thanks! What kind of error do you get with Enzyme? |
EnzymeAD/Enzyme.jl#2794 might be a smaller MWE. |
|
just for fun what if you use Enzyme from within reactant (the sret stuff above doesn't matter there) |
|
I don't know what that means :) |
I dusted off my four-year-old https://github.com/KristofferC/HyperHessians.jl, and it seems to still be quite good. Benchmarks run on 1.10 due to JuliaLang/julia#60241 and JuliaLang/julia#60243.
The package has had very little real world use but seems to work here at least and be quite a bit better.