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

Slight persistence patch and HTMObjectIn/Output constructor change. #419

Merged
merged 6 commits into from
May 2, 2016

Conversation

cogmission
Copy link
Collaborator

Fixes #417
Fixes #418

Support not halting the Network when serializing it during non-threaded use. This support is built in to the checkpointing mechanism so halt doesn't have to be called, but in the case where a library has its own checkpointing mechanism (i.e. Flink), this patch will support that use case if the Network is not in threaded mode.

If the Network is in threaded mode then users are forced to use the built in checkpointing mechanism until such time as a need develops to allow some flexibility with that option.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 80.931% when pulling 6693834 on cogmission:slight_persistence_patch into 95ddd4c on numenta:master.

@cogmission
Copy link
Collaborator Author

@rhyolight I want to merge this when coveralls stops complaining - its an update needed by @EronWright for flink-htm. I added very minor changes...

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 81.048% when pulling 22ba2d7 on cogmission:slight_persistence_patch into 95ddd4c on numenta:master.

@cogmission
Copy link
Collaborator Author

@EronWright Do you want to give this a go before I merge it?

@EronWright
Copy link
Contributor

@cogmission I tried it out and it looks good to me.

@cogmission cogmission merged commit 4684f8b into numenta:master May 2, 2016
@cogmission cogmission deleted the slight_persistence_patch branch July 23, 2016 11:52
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.

3 participants