David Ray
David Ray
`column` should be `columnIndex` [See here](https://github.com/numenta/nupic/blob/master/src/nupic/research/spatial_pooler.py#L685)
I think this is a hold-over from before the stimulusThreshold enforcement, that never got reconsidered? The list returned is the [list of active columns](https://github.com/numenta/nupic/blob/master/src/nupic/research/spatial_pooler.py#L777) (in sparse form), and is used...
The SpatialPooler's `potentialRadius` "silently" gets adjusted in the background during initialization of the OPF or Network API, to be the same size as the `inputWidth` (as opposed to its default...
It appears this is no longer used, but the variable still exists and is getting serialized for nothing - it looks like? https://github.com/numenta/nupic/blob/master/src/nupic/research/spatial_pooler.py#L330 It used to exist in the `inhibitColumns()`...
`temporal_memory_test.py` [successfully creates a segment here](https://github.com/numenta/nupic/blob/master/tests/unit/nupic/research/temporal_memory_test.py#L636) on a `Cell` which doesn't exist according to the parameters set on the TemporalMemory both [here](https://github.com/numenta/nupic/blob/master/tests/unit/nupic/research/temporal_memory_test.py#L618) and [here](https://github.com/numenta/nupic/blob/master/tests/unit/nupic/research/temporal_memory_test.py#L619). According to these, there should only...
A small optimization for clarity's sake and optimal performance: The [if clause here](https://github.com/numenta/nupic/blob/master/src/nupic/research/connections.py#L423) should be removed and the appending of the synapse should be added above in the `else` clause....
Found a slight (but important) oversight. [This for loop](https://github.com/numenta/nupic/blob/master/src/nupic/research/connections.py#L323) should have a `break` statement on line 327. This will avoid looping through all of the segments after a Segment marked...
Knn dev
Fixes #132 **Work In Progress**... (i.e. Merge not yet requested) - [x] Create new Parameters for application to KNNClassifier construction - [x] Create new Constants class to begin getting rid...
SEE: https://github.com/numenta/nupic/issues/3653 Time to sync the TemporalMemory and test class again...
The Map holding encoder parameters contains a key which repeats values on a nested level. The nested (child) level designated by the key ```fieldEncodings```is the one actually used (and not...