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

Add the Co-Information #10

Closed
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@Autoplectic
Copy link
Member

Autoplectic commented Oct 4, 2013

aka the multivariate mutual information.

@Autoplectic

This comment has been minimized.

Copy link
Member Author

Autoplectic commented Oct 5, 2013

that was dangerous -- travis config needs updating. the package wasn't getting installed because i didn't have iterutils set as an install package, but nosetests passed in the root directory. check out the build log of commit ff1e0fc .

@chebee7i

This comment has been minimized.

Copy link
Member

chebee7i commented Oct 5, 2013

Yeah the issue was actually with coverage. The coveragerc file was set not to raise on ImportErrors. I caught it on my system because I didn't have iterutils installed. This is all fixed now, on my fork. I'll merge it in soon.

@Autoplectic

This comment has been minimized.

Copy link
Member Author

Autoplectic commented Oct 6, 2013

this has been merged outside of this pull request.

@Autoplectic Autoplectic closed this Oct 6, 2013

@Autoplectic Autoplectic deleted the Autoplectic:coinformation branch Oct 6, 2013

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