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

Fixing numerical issue and adding utility function #292

Merged
merged 4 commits into from Nov 7, 2017

Conversation

Projects
None yet
2 participants
@cbaldassano
Collaborator

cbaldassano commented Nov 7, 2017

Changed the transition probability to scale with the number of timepoints, which should not change the solution (all valid paths have the same number of transitions) but gives better numerical stability. Also, added a utility function for computing weighted variance, which will be useful for the notebooks I'll be releasing in the future for the Schema project.

@mihaic

mihaic approved these changes Nov 7, 2017

@mihaic mihaic merged commit dcda29a into brainiak:master Nov 7, 2017

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
linux Build finished.
Details
macos Build finished.
Details

danielsuo pushed a commit that referenced this pull request Nov 16, 2017

Fix bug in redis module tests. (#292)
* Fix bug in redis module tests.

* Sleep while waiting for next message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment