Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Fix destruction of synapses when destroying segment #3880

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dan-mk
Copy link

@dan-mk dan-mk commented Aug 14, 2019

When the predictedSegmentDecrement TM parameter is set to some value greater than 0, I was getting this error:

Captura de tela de 2019-08-07 02-42-00

Using the destroySynapse function prevents the error in my tests.

@rhyolight
Copy link
Member

@dan-mk Can you update the test suite to fail when predictedSegmentDecrement is non-zero? It would be best to expose this as an error in an existing test suite.

@rhyolight
Copy link
Member

BTW @dan-mk, this repository is in maintenance mode. There is no guarantee this contribution will be merged anytime soon. See the README for details. The currently active HTM C++/Python project is https://github.com/htm-community/htm.core

@dan-mk
Copy link
Author

dan-mk commented Aug 21, 2019

@rhyolight I'm not sure about the right way to contribute to the test suite, so I'm attaching a modified "hello_tm" that should show the error before 60k iterations. If it does not, I've probably done something wrong

Captura de tela de 2019-08-21 03-30-30

hello_tm.zip

I understand this repository is in maintenance mode and that it's not a critical bug (since it seems like I'm the first one having problems with this), so no problem if it's not merged

I'm also aware about the community repository, but I had the same error with its code, except that it gave me a segfault which is not as easy to debug haha

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

Successfully merging this pull request may close these issues.

2 participants