-
Notifications
You must be signed in to change notification settings - Fork 6
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
documentation has the wrong format for fftsettings #42
Comments
and for all maxParameter actually... |
maybe this should be in the docs repo, @weefuzzy ? |
(and I disagree with my first comment - I think removing them (as they cannot be dynamic) is probably better. so removing from the doc and removing from the interface as they are redundant. |
When you have time could you flesh out the original issue description a bit? I honestly have no notion what the details refer to, even if I once did. But, yes, if it's a documentation problem, we should also transfer to docs |
Sorry, here goes. At the moment, the generated doc has for the maxParam both Param and maxParam fftsettings and maxfftsize yet the interface is requiring to bundle the max at the end (or a minus one) or we get an error. so what would be amazing:
|
we should either remove the pad options (bad) or add it in the doc (better)
The text was updated successfully, but these errors were encountered: