-
Notifications
You must be signed in to change notification settings - Fork 20
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
confusing lack of full name for StdRng on guide-rngs.html #58
Comments
ChaCha12 should be faster... those benchmarks probably are a little fishy. We previously used ChaCha20 (and before that HC-128, and before that Isaac RNG), so we have changed the generator while roughly retaining those properties — except for improving memory usage and init speed. Also, there are unresolved questions (#57) and the book probably does have some flaws. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The table in https://rust-random.github.io/book/guide-rngs.html#cryptographically-secure-pseudo-random-number-generators-csprngs lists StdRng with known properties for "performance", "initialization", "memory", "security (predictability)", "forward secrecy", but notably does NOT give its full name: ChaCha12. This is confusing.
Like, what are we trying to do here? If we don't want to tie the choice of RNG down to ChaCha12, we would have no definite answers for "performance", "initialization", "memory", etc. either. If we have definite answers for these columns, it naturally follows that we definitely know what generator it is using.
While we're at it, it's a little fishy that ChaCha12 is running slower than ChaCha20. Is this a fluke in the benchmark, outdated table, or some real performance issue?
The text was updated successfully, but these errors were encountered: