neuronsimulator / nrn

NEURON Simulator
http://nrn.readthedocs.io
Other
406 stars 118 forks source link

Refactor `src/gnu` #1330

Open alexsavulescu opened 3 years ago

alexsavulescu commented 3 years ago

Not familiar with all the code in gnu (except for the d_avec refactoring to std::vector) A couple of ideas to brainstorm:

nrnhines commented 3 years ago

I'd be delighted to see the gnu subdirectory disappear.

alexsavulescu commented 3 years ago

Was discussing today about PRNGs, we were wondering if we shouldn't drop some unused number generators?

https://nrn.readthedocs.io/en/latest/python/programming/math/random.html?highlight=random#Random.ACG

As far as I understand , only Random123 and MCellRan4 are of interest wrt support for parallel simulations. Do we have any usecase for the rest? If so, I imagine we can use std implementations for most of them (readily available)

cc: @jamesgking

ramcdougal commented 3 years ago

For what it's worth, MLCG and ACG do not appear to be used in any ModelDB entries.