Jannik Luboeinski
Jannik Luboeinski
Yeah, that would be great!
Thanks @noraabiakar, the description looks good to me! Meanwhile, I implemented the stochastic term in the C++ code compiled from NMODL, as we had discussed. I defined the stochastic variable...
I just wrote [in issue #1643](https://github.com/arbor-sim/arbor/issues/1643#issuecomment-944434427) about a current workaround using the Euler method and the `minstd_rand0` random generator of the C++ standard library. @noraabiakar What are the arguments in...
Thank you for your answer, Ben. For my purposes, `minstd_rand0` with careful seeding worked fine, so far. But I see that for general use in arbor Random123 will be much...
> Is this addressing both #1643 and #1655 ? I guess so!?
I think it would be nice to have the opportunity to keep low the memory usage by loading only the data of a defined number of timesteps back in the...
For the record: Unfortunately, the suggestion by @schmitts doesn't solve the whole issue. We've found that there might be a general issue with long timesteps: 1. Arbor uses interpolation for...
Okay, `black` now passes (I had previously used an older version, seems that they changed something). Yeah, now `flake8` is complaining... I'll try to fix that, too :)
> Hey, made some comments for improvements, rest looks great, thanks. Thanks @thorstenhater! I've revised the code according to the comments.
The different-radii and different-length tests are now uncommented. They should succeed if they are run with this new fix in the diffusion solver: https://github.com/thorstenhater/arbor/commit/7577092f89e9b54d8fde26ac6963a7c9965e43cc