-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fixing spatial anomaly for htmcore WIP #15
base: master
Are you sure you want to change the base?
Conversation
HTMcore without "spatial anomaly" and with "synapse competition":
EDIT: |
Yea I don't know what to say here. Numenta did several non-biological things in order score higher on the NAB benchmark. I think we should disregard both the spatial anomaly and the backtracking TM, since both are non-biological methods which improve scores on this particular benchmark (but probably not all benchmarks). |
yep, we're catching up to "HTM" with backTM, even Numenta lists the
definitely we should. I was really worried even Numenta's detector suffer the problem (most work done by "spatial anomaly"), fortunately it is not a problem (so the NAB paper is valid!), unfortunately, our TM suffers w/o the spatial anomaly. |
This reverts commit 354a025.
which is "fake" and non-biological, not HTM related. Unfortunately, with the current settings the code affected the results the most.
c40c217
to
d6023e3
Compare
@ctrl-z-9000-times I'll try to revisit this again, the issue is a weird/severe performance regression. Please help me investigating if you have time, thanks |
Update:
|
This reverts commit e34e6f4.
I merge it and i got
It is the "good" score? I can merge it with this branch if you like |
I don't have write permissions for this NAB repo. The branch is there: https://github.com/Zbysekz/NAB/tree/fixing_spatial_anomaly |
Now i got even better: (assume it is because of seed==0)
|
Previous was with spatial anomaly=True, with false it is:
Not sure if it should be set to true or false in this branch? |
thank you 👍
that was slightly below good :), I think I got same-as Numenta scores. (hope with correct settings too)
we want spatial=False here, and everywhere. |
Fixing spatial anomaly
@Zbysekz btw, the htmcore detector was not in your PR, so the conflict is still there |
Yes, it would be not possible anymore. I coudn't keep Panda in this way. With using NAPI, everything is more general and generated automatically in contrast of manually handing over specific data & instances. But i can rewrite this nab detector to use NAPI, what i saw it should be quite simple. |
Hmm there is a bit complication, because i wanted to revert pandaVis changes, but these are not in this branch yet so that is why it was not reflected... i will start new PR and we will just link it from here |
I was thinking about it...let's keep it for a while as is, and if we stabilize the (top) scores, we can rewrite to NAPI. What I wonder is overhead of the NAPI, for c++->py it's about 30%. |
What you mean by this overhead of 30% ? |
compute speed overhead. |
Undo panda vis and resolve merge conflict in fix_spatial
Hmm yes 30% overhead is good, i would expect more. |
I am getting
without spatial anomaly and with latest htm.core |
Hi @Zbysekz! Could I possibly see your implementation at all? Thanks, Sam |
Hey @breznak! Could I possibly see your implementation that got these scores if you still have it?? Sam |
Hello Sam, What about this? "good as Numenta results without the fake spatial anomalies" as Breznak writes in one comment. I thnik we discuss this somewhere, but it is already so long ago. I am not i this project active right now. Good luck |
Hi @Zbysekz! |
No i just ran latest htm.core with this branch (that is not merged to NAB master) with spatial anomaly off.. |
All good man thank you! |
Hi @ctrl-z-9000-times, yes thank you that'd be awesome! |
Ok @gotham29, you should now have write permission for this repo. |
So, we have a problem.. hopefully a combination of the params for HTMcore is not optimal.
The Numenta detectors have an "artificial" way of detecting "spatial anomaly" (which is a running upper/lower bounds threshold detector)
If I disable such (fake) detector for our
htmcore
the scores become pretty bad:This means most of the work in htmcore with these settings is done in the threshold detector.
Luckily, the numenta detectors & settings do not suffer this drop when spatial-threshold is removed.
TL;DR: we must optimize our net without this "spatial anomaly" threshold-detector.
For #6