-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Fourth order EPIRK methods #410
Changes from all commits
7a4fcf6
f0b06cf
144e395
8486f91
461da92
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,8 +49,8 @@ end | |
prob = ODEProblem(f, u0, (0.0, 1.0)) | ||
prob_ip = ODEProblem{true}(f_ip, u0, (0.0, 1.0)) | ||
|
||
dt = 0.1; tol=1e-5 | ||
Algs = [Exp4] | ||
dt = 0.05; tol=1e-5 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. was there an issue here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you recall our numerical experiments on Using the previous Now that you mentioned it, comparing the results to Tsit5 is not a good test and should probably be rewritten using There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh I just meant the We don't need analytical solutions for convergence testing, though we should probably switch to using a better analytical solution when SciML/DiffEqProblemLibrary.jl#8 is there. @Vaibhavdixit02 did you ever add the analytical solution? SciML/SciMLSensitivity.jl#14 |
||
Algs = [Exp4, EPIRK4s3A, EPIRK4s3B] | ||
for Alg in Algs | ||
gc() | ||
sol = solve(prob, Alg(); dt=dt, internalnorm=Base.norm, reltol=tol) | ||
|
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.
The EPIRKs should probably use the standard route for grabbing Jacobians, or everything should be updated at the same time.
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.
It's fine, keep it. SciML/DiffEqBase.jl#113 makes DiffEqFunction-based ODEProblems the default, so this'll be the new style.