-
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
Draft of issues fixed #129
Conversation
CHANGELOG.md
Outdated
[26](https://github.com/haskell/random/issues/26) | Add Random instances for tuples | | ||
[44](https://github.com/haskell/random/issues/44) | Add Random instance for Natural | | ||
[51](https://github.com/haskell/random/issues/51) | Very low throughput | | ||
[53](https://github.com/haskell/random/issues/53) | incorrect distribution of randomR for floating-point numbers | Fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[53](https://github.com/haskell/random/issues/53) | incorrect distribution of randomR for floating-point numbers | Fixed | |
[53](https://github.com/haskell/random/issues/53) | incorrect distribution of randomR for floating-point numbers | Documented behaviour of floating point numbers |
We haven't actually done this yet :) I'll create a PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooh yes we meant to do it - thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've started writing a little Haddock section for this. I'll create a PR once I have a draft.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I've suggested a few comments to give context / describe how we have fixed various issues.
I might squash all the commits before making a PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one Markdown formatting nit - otherwise this is great, thank you!
Edit: I just committed my formatting suggestions to the branch in dd39abb.
I think that would be a good idea :) |
dd39abb
to
37f2271
Compare
No description provided.