Simplify interaction of random number generators and lazyarray #553
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There are two changes here:
(1) previously, a random number generator with
parallel_safe=False
would always draw a reduced number of values when run with >1 MPI processes, according to the number of processes, unless themask_local
parameter was set toFalse
. Now, a mask must be explicitly provided if you want to draw a reduced number of values (i.e. only those values consumed on that node),mask
(renamed frommask_local
) is either an array, or None, and can no longer take the valueFalse
.(2) moved the lazyarray behaviour for
RandomDistribution
from the lazyarray package to PyNN, to keep all the rng-related logic in one place.