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.

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

Random numbers help files #40

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

MattS6464
Copy link
Contributor

Updated help files and added audio examples to randfloats, randomvals and randomvals~

@AlexHarker
Copy link
Owner

AlexHarker commented Aug 13, 2024

ahonise~ uses a different RNG to the one stated. Details are:

A complementary modulo with carry algorithm (proposed by George Marsaglia)
Details can be found in Marsaglia, G. (2003). "Random number generators". Journal of Modern Applied Statistical Methods 2
See - http://digitalcommons.wayne.edu/cgi/viewcontent.cgi?article=1725&context=jmasm

Suggest only the first line for the object description, but perhaps the details for the xml if there is somewhere to put it (this is in the source, but wasn't updated in the helpfile so you weren't to know)

@AlexHarker
Copy link
Owner

The above applies to all four objects.

Also:

  • could the webarchive link break at a better point somehow (resize the comment box)
  • across multiple patches there's a bpatcher with the phrase "reset after changing parameters)" - there's a missing opening bracket on that
  • across multiple objects the gaussian input should be named "list" not "message" or "gaussian distribution"
  • randomvals~ has an "input" message which should be in the signal category
  • randomvals should have an "int" message and not "input"

@MattS6464
Copy link
Contributor Author

Corrected the link to the RNG source. Changed xml files to correctly show method names. Tided issues in help files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants