etaler / Etaler

A flexable HTM (Hierarchical Temporal Memory) framework with full GPU support.
BSD 3-Clause "New" or "Revised" License
89 stars 14 forks source link

SpatialPooler constructor confusion #49

Closed marty1885 closed 5 years ago

marty1885 commented 5 years ago

So, this problem shows up when I'm building my final project for my univ.

A SpatialPooler is designed to be easily construct-able. And the default format of the constructor is: SpatialPooler(input_shape, output_shape). So, when you call the constructor like so, SpatialPooler({128}, {32}) you'll expect a SP that takes in 128 bits and generates a 32bit output. But no, in fact this calls to the alternative constructor for topology. i.e. SpatialPooler(input_shape, kernel_size).

This is super confusing, We'll have to fix this.