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.

Sign up for GitHub

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

Jump to bottom

Misleading comment in SP #892

Closed
chetan51 opened this issue Apr 30, 2014 · 2 comments · Fixed by #895
Closed

Misleading comment in SP #892

chetan51 opened this issue Apr 30, 2014 · 2 comments · Fixed by #895
Assignees
@chetan51
Labels
subject:algorithms
Milestone
Sprint 21

Comments

@chetan51
Copy link
Contributor

In the SP's adaptSynapses, a comment reads: "Permanence values for shared inputs, which are input bits that are turned on and connected to more than one active column, are decreased in order to discourage multiple columns from learning the same input pattern."

This is not actually being done, so we should update the comment.
@chetan51 chetan51 added this to the Sprint 21 milestone Apr 30, 2014
@chetan51 chetan51 self-assigned this Apr 30, 2014
@breznak
Copy link
Member

breznak commented Apr 30, 2014

Sounds relatively smart though, maybe we should make a new issue to
re-implement this.

@chetan51 chetan51 mentioned this issue Apr 30, 2014
Closed
@chetan51
Copy link
Contributor Author

@breznak Great idea, I've made a new issue here: #894

@chetan51 chetan51 mentioned this issue Apr 30, 2014
Merged
@subutai subutai mentioned this issue Jul 20, 2016
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chetan51 chetan51

Labels
subject:algorithms
Projects
None yet
Milestone
Sprint 21
Development

Successfully merging a pull request may close this issue.

Removed misleading comment chetan51/nupic
2 participants
@chetan51 @breznak