-
Notifications
You must be signed in to change notification settings - Fork 33
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
Replace random number generator? #194
Comments
On further reading, O'Neill doesn't seem to like Xoroshiro. He has some interesting blog posts. |
I've downgraded this as I think the current code is not broken and there seems to be no overwhelming reason for changing the random number generator. I would want to know what is the potential saving in compute time. |
I noticed in one of Theresa's scenarios that the random number generator took about 40% of the compute time; this was because of the extreme use of many intervention components and some "odd" behaviour (check coverage before checking whether component meets other conditions). Probably in other scenarios performance would not be massively affected; I can't actually measure impact without implementing the change however. |
This is a sub-set of #204. |
This was done (at least partially) in #242. |
OpenMalaria uses the Mersenne Twister 19937 random number generator. I am not an expert on the topic but have heard that although the generator is reasonably good it has some "bad states" from which it takes a while to escape and sequences yielded by nearby seeds may take a while to diverge.
The best fast non-cryptographic RNGs I am aware of are Xoroshiro and PCG. Fast cryptographic generators may also be of interest, e.g. ISAAC.
The text was updated successfully, but these errors were encountered: