Add Local Regularization Support#1016
Conversation
|
Missing tests?
You can instead |
3b6d5d9 to
f49f3a2
Compare
|
Sorry for the delayed response. I was going through final exams over the past three weeks. I rebased and resubmitted the PR on top of the latest DiffEqFlux master. I updated the implementation so that it no longer requires any changes to OrdinaryDiffEq.jl. All changes are now contained in this DiffEqFlux PR. Following your suggestion, the forward local regularization path now calls For the backward pass, I added a custom rrule so that the pullback does not differentiate through the mutable integrator state updated by I will add tests soon. For now, I ran a 3000-step MNIST ODE experiment on the AMD Radeon RX 7900 XT with Julia 1.12.5, following the parameter settings from the paper. The results are:
|
Here are the latest changes
The modifications to SciMLBase are no longer needed now. I have implemented memory reuse to improve speed. Additionally, for the
biasedpart, I used reservoir sampling to reduce memory consumption (this should be mathematically equivalent to the original implementation?)This PR is to solve issue#817.
I have completed implementing the
perform_stepfor an ODE solver — Tsit5, and I ran a 3000-step MNIST ODE experiment on the AMD Radeon RX 7900 XT following the parameter settings from the paper. The results are as follows:I placed the implementation of
perform_stepinOrdinaryDiffEq.jl/lib/OrdinaryDiffEqTsit5/src/tsit_perform_step.jl. I am not sure if it would be appropriate to put it directly in DiffEqFlux.Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.