-
Notifications
You must be signed in to change notification settings - Fork 484
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
Added functions for generating uniform/normal distributed randomness #154
Conversation
…cting with `no_extension_access_modifier` rule
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.
Great work, @regexident! Only question I had was about the naming of randUniform
& randNormal
. Would it make sense to expand to randomUniform
& randomNormal
for clarity? Or is the use of rand
in keeping with other well-known abbreviations whose use is less conspicuous than expanding to a more descriptive name (e.g. det(_:)
-> determinant(of:)
)?
Yeah, I was keeping it short, to blend in with the rest of Surge. I'm 50/50 between |
Let's go for |
49ebd0d
to
9b93025
Compare
As this is a purely additive change I figured we actually don't need to wait for a v3.x.
We have plenty of stuff on our plate for that already.
Resolves #144