Skip to content

The RandomDistributedScalarEncoder needs ability to set numBuckets  #443

Open

Description

@cogmission

numBuckets needs an official Parameter in Parameters.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.

Activity

  1. rhyolight commented on Jul 25, 2016

    @rhyolight
    Member

    So will Parameters have access to the input data to calculate the min / max values required to calculate the resolution?

  2. cogmission commented on Jul 25, 2016

    @cogmission
    CollaboratorAuthor

    I don't believe this is necessary. Alex mentioned that they are "auto-configured" from the numBuckets parameter... as seen here and here (numBuckets == maxBuckets, deeper in the code)

    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?

  3. cogmission commented on Jul 25, 2016

    @cogmission
    CollaboratorAuthor

    For instance, the NAB assumes that the numBuckets parameter is sufficient and doesn't specify resolution specifically, however HTM.Java doesn't have this so it thinks the resolution parameter is missing and throws java.lang.AnnoyingAsHellException :-P

    My fix for this was to add resolution to the NAB JSON config file, but if we were going to do it the NuPIC way, we would allow the numBuckets parameter to be picked up and usher in the configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions