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

Encoders using vector input like CoordinateEncoder do not work anymore #3753

Closed
rhyolight opened this issue Aug 8, 2017 · 0 comments
Closed

Comments

@rhyolight
Copy link
Member

I found this issue when trying to get https://github.com/numenta/nupic.geospatial running with NuPIC 1.0.

Looks like this is the origin of the bug:

34dd874#diff-d62e7e036bc450e0076245d0839b7ef4R381

In this commit, we start using self.predictedField as a condition to get encoder outputs instead of self.predictedFieldIdx. The CoordinateEncoder's predictedFieldIdx is always -1, so it worked before the change. But the CoordinateEncoders' predictedField is not None, it is vector, so the compute function tries to get bucket values from the encoder that do not exist.

We need to add a special condition to the compute function to check also for self.predictedField == "vector".

@rhyolight rhyolight self-assigned this Aug 8, 2017
@rhyolight rhyolight changed the title Predicted field is not an integer when using coordinate encoder Encoders using vector input like CoordinateEncoder do not work anymore Aug 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant