Remove FlatSpatialPooler #2056
Remove FlatSpatialPooler #2056
Conversation
@@ -1789,6 +1789,56 @@ def testWrite(self): | |||
self.assertSetEqual(indices1, indices2) | |||
|
|||
|
|||
def testRandomSPDoesNotLearn(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test was ported from FlatSP and works fine.
5ca46e2
to
70531a6
Compare
70531a6
to
d02ebb3
Compare
CC @chetan51 |
@chetan51 Ping... |
👍 |
…lPooler Remove FlatSpatialPooler
|
||
def compare(self, pySp, cppSp): | ||
self.assertAlmostEqual(pySp.getNumColumns(), | ||
cppSp.getNumColumns()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this c++/py compatibility test should stay! (ofc ported to the new SP implementation, not FlatSP)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this c++/py compatibility test should stay! (ofc ported to the new SP implementation, not FlatSP)
Relax, man.. 😌 Look these here:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool then! 😎
Sorry, I just saw the files removed in the recent pull, and none tests added...
I'm raising a finger of caution on the removed tests, thoughts @david-ragazzi @chetan51 ?
|
…tSpatialPooler Remove FlatSpatialPooler
Fixes: #627