Skip to content

Update README.txt#48

Open
D-McDougall wants to merge 1 commit into
masterfrom
update-score-for-HTM.Core
Open

Update README.txt#48
D-McDougall wants to merge 1 commit into
masterfrom
update-score-for-HTM.Core

Conversation

@D-McDougall

Copy link
Copy Markdown
Collaborator

As you can see, the scores went the wrong way.

I think part of the issue is the parameters for HTM-core are not great.

@breznak breznak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, this is really worse. Hope we can tune htmcore's params to be competitive

@D-McDougall

D-McDougall commented Sep 30, 2021

Copy link
Copy Markdown
Collaborator Author

Yes, I think we definitely can do much better than these low scores...

I think some of the parameters got changed at some point; I don't think they match what numenta was using originally.

IIRC, Original Parameters are here: https://github.com/numenta/nupic/blob/master/src/nupic/frameworks/opf/common_models/anomaly_params_random_encoder/best_single_metric_anomaly_params_cpp.json

Here is the output of my new nyc_taxi.py program:

broken_tm

The TM is clearly not working. The blue line is the raw anomaly, which is almost always at 100%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants