djns99 / CUDA-Shuffle

MIT License
8 stars 1 forks source link

Add Philox #6

Closed RAMitchell closed 3 years ago

RAMitchell commented 3 years ago

Paper: http://www.thesalmons.org/john/random123/papers/random123sc11.pdf Reference implementation: https://github.com/DEShawResearch/Random123-Boost/blob/master/boost/random/philox.hpp

RAMitchell commented 3 years ago

figure0 Speed looks good and all tests are passing with 16 rounds.

djns99 commented 3 years ago

Looks good. Does it pass the https://github.com/djns99/CUDA-Shuffle/blob/master/test/ParameterQualityTests.cu tests? Those are the ones currently included in the repo for quality validation

djns99 commented 3 years ago

@RAMitchell looks good. Is this ready to be merged?

RAMitchell commented 3 years ago

Yep.