-
Notifications
You must be signed in to change notification settings - Fork 160
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
The RandomDistributedScalarEncoder needs ability to set numBuckets
#443
Comments
So will |
I don't believe this is necessary. Alex mentioned that they are "auto-configured" from the I'm not averse to having encoder params at all. We do it for the DateEncoder (due to it's need for varied configuration), so we can do it for others as well? |
For instance, the NAB assumes that the My fix for this was to add |
numBuckets
needs an official Parameter inParameters.java
and it needs to be set by the Network API upon initialization.Care must be taken to guarantee that the
resolution
is properly calculated and that the param is tested when absent as well.The text was updated successfully, but these errors were encountered: