-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
Weak adaptive solvers give rise to Out-of-memory errors #427
Comments
@frankschae is there a good way to avoid allocating the matrix of all pairwise random numbers there? |
yeah we never use the full matrix |
With StochasticDiffEq v6.36.0 it should be working, right? Unfortunately, I'm still getting the same error. |
It can't be the same error because the line it was pointing to doesn't exist anymore. Maybe similar error different stack trace? |
|
hmm that one looks worse. It's pointing to: g2 = [zero(noise_rate_prototype) for k=1:m]
I don't see how one could change that to avoid the m stored copies except by tons of recomputations (because g2 and g3 are used at several stages). Do you have any simplifications of your model with respect to g ? In your first code on discourse, you were using DRI1NM which doesn't have this issue because it assumes that the diagonal components do not mix.
|
Yes, tried it earlier and it gives a similar error. didn't read throuth the stack trace because I assumed it was the same, but apparently that's not the case
|
Interesting.. and
which is just the third cached noise vector. SOSRI() should have (up to the Butcher table) exactly the same cache consumption:
and SOSRA() as well (-one vector of size m for the random numbers):StochasticDiffEq.jl/src/caches/sra_caches.jl Line 260 in a4497c5
|
Yes, with SOSRA I got |
See https://discourse.julialang.org/t/why-does-my-devectorized-code-keep-allocating-memory/62727/11
For a set of Order 1e4 coupled SDE equations with additive diagonal noise, SOSRA and similar works fine, but the DRI-methods crash with OutOfMemoryError()
Apparently, this is "because Ihat2 uses length(u)^2 memory for the pairwise 3-point random numbers"
The text was updated successfully, but these errors were encountered: