-
Notifications
You must be signed in to change notification settings - Fork 2
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
Coordination: changes to PRNG libraries #23
Comments
https://github.com/bos/mwc-random/compare/master...curiousleo:random-v1.2-compat?expand=1 fails to build with
I don't know how to fix that. |
@curiousleo It is a |
Fixed. Thanks! |
Should we close this or use it to track
|
Alright gentleman, I implemented some benchmarks for all pure RNGs available in Haskell and results are pretty impressive. Here is a PR and a branch that you can use to run benchmarks for both random-1.1 and random-1.2: lehins/haskell-benchmarks#7 Here are also all the repo's with fixed
It is pretty cool that changes were incredibly minimal and all of them are backwards compatible. To run benchmarks:
And so on for other types. (It's possible to run them all at once, but I find the plots bit noisy that way) Next in line are stateful RNGs, but those I think I'll do some time later. |
This is fantastic!
I would also be interested in benchmarks for |
To avoid duplication of work, let's track work we've done to make PRNG libraries compatible with the
interface-to-performance
branch here.Feel free to edit this description directly.
splitmix
: https://github.com/lehins/splitmix/tree/new-random (diff)RandomGen
Prim
pcgen
: https://github.com/curiousleo/pcgen/tree/new-random (diff)RandomGen
Prim
mwc-random
: WIP https://github.com/curiousleo/mwc-random/tree/random-v1.2-compat (diff)MonadRandom
random-fu
: does not depend onrandom
; usesrandom-source
to abstract over concrete PRNGsQuickCheck
: builds withStdGen
fromrandom v1.1
on non-GHC platforms. That won't work withrandom v1.2
.hedgehog
: requiresinstance UniformRange Integer
which is currently missing.random v1.2
random v1.2
: missinginstance UniformRange Integer
The text was updated successfully, but these errors were encountered: