Skip to content
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.

Sign up for GitHub

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

Jump to bottom

rand::random's distribution and seediness should be documented #259

Closed
njaard opened this issue Feb 13, 2018 · 1 comment
Closed

rand::random's distribution and seediness should be documented #259

njaard opened this issue Feb 13, 2018 · 1 comment
Labels
C-docs Documentation

Comments

@njaard
Copy link

njaard commented Feb 13, 2018

the function rand::random()'s behaviors are insufficiently documented:

  • What is the distribution (especially with ::<f32>)?
  • It's strongly implied that this function is automatically randomly seeded at each invocation, but it would be better to say so outright.
@dhardy
Copy link
Member

dhardy commented Feb 14, 2018

Currently this uses Rand for distributions. But, sure, I understand that the doc could be better!

@dhardy dhardy added the C-docs Documentation label Feb 14, 2018
@dhardy dhardy closed this as completed in d82bad1 Feb 15, 2018
pitdicker pushed a commit that referenced this issue Apr 4, 2018
@dhardy
Improve doc for thread_rng and EntropyRng.
70e7e92 
Fix #259.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-docs Documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dhardy @njaard