Skip to content
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

EM() as the default solver? #21

Open
oameye opened this issue Feb 21, 2024 · 3 comments
Open

EM() as the default solver? #21

oameye opened this issue Feb 21, 2024 · 3 comments
Labels
question Further information is requested

Comments

@oameye
Copy link
Member

oameye commented Feb 21, 2024

Why do we use EM() as the default solver? From the example of StochasticDiffEq.jl docs this method seems to be prone to errors. The same can be said for Euler() as the deterministic solver.
image

@reykboerner
Copy link
Collaborator

Indeed Euler-Maruyama can be inadequate especially for large step sizes, but it is still so commonly used that one could argue it is still a reasonable default solver. Users would potentially be less familiar with the other SDE solvers. Which one would you use?

For ODEs I argee that one could replace Euler by something like RK4() or DP5().

@reykboerner reykboerner added the question Further information is requested label Mar 13, 2024
@oameye
Copy link
Member Author

oameye commented Mar 15, 2024

DP5() is dangerous as it isn't useful for damped systems. I would use the default solvers that SciML uses which would be Tsit5() and SRIW1() I think.

@reykboerner
Copy link
Collaborator

Hmm, Tsit5() caused some problems when used with DynamicalSystems.jl (see this issue). But I agree that it's best to go with established defaults if we don't stick with the most basic defaults.

@oameye oameye added this to the Release version 1.0 milestone Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants