Skip to content
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

Document better the new Extended TM parameters apical/basal input dim #1135

Open
breznak opened this issue Oct 23, 2016 · 2 comments
Open

Document better the new Extended TM parameters apical/basal input dim #1135

breznak opened this issue Oct 23, 2016 · 2 comments

Comments

@breznak
Copy link
Member

breznak commented Oct 23, 2016

With the new update, I found ETM requires new initialization parameters:
https://github.com/numenta/nupic.core/blob/master/src/nupic/experimental/ExtendedTemporalMemory.hpp#L96

There's no documentation for them, nor reasonable defaults... I searched in examples and tests.
Please provide a better explanation & suggested default values.

@rhyolight
Copy link
Member

@subutai @mrcslws I would like to clarify the line between research and production code. I see there is now an experimental folder in nupic.core, but not in nupic.

I can understand why you might do this to do C++ research without adding the make complexity to nupic.research. Is that the case here? If so, I think you should an experimental/README.txt that clearly defines this directory as research with the normal disclaimers.

@mrcslws
Copy link
Contributor

mrcslws commented Oct 31, 2016

Yeah I agree that the research code could be better separated from the public API.

(FYI there is already a experimental/README.md, but maybe it should be longer.)

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

No branches or pull requests

3 participants