[Chapel Merge] Add support for fillRandom with bounds

Notifications Commits
1

Branch: refs/heads/main
Revision: f5a0d97
Author: mppf
Link: Add support for fillRandom with bounds by mppf · Pull Request #19540 · chapel-lang/chapel · GitHub
Log Message:

Merge pull request #19540 from mppf/pcg-fillrandom-min-max

Add support for fillRandom with bounds

Requested in Closes #1174 ak.randint() fails for ranges greater than 2**63 by Ethan-DeBandi99 · Pull Request #1214 · Bears-R-Us/arkouda · GitHub

  • adds PCGRandomStream.fillRandom with min, max arguments for the bounds
  • adds PCGRandomStream.iterate with min, max arguments for the bounds
  • changes proc shuffle to use getNext_noLock rather than
    randlc_bounded because randlc_bounded has more complex calling
    requirements (namely the position must be passed). There isn't a good
    reason for it to track the position at all in this code so this is a
    simplification and one that it is easier to write correctly.

Reviewed by @lydia-duncan - thanks!