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

[coor|tica|variational] check for rank zero correlation matrices #1055

Merged
merged 1 commit into from
Mar 1, 2017

Conversation

fabian-paul
Copy link
Member

fixes #1053
Comments and suggestions welcome.

@marscher
Copy link
Member

marscher commented Mar 1, 2017

great! Thank you.

@fabian-paul
Copy link
Member Author

The tests fail because of this doctest (?)
https://github.com/markovmodel/PyEMMA/blob/devel/pyemma/__init__.py#L51

@marscher
Copy link
Member

marscher commented Mar 1, 2017

It is IT maintainence day and therefore the file needed in this test can not be downloaded 👍

@marscher
Copy link
Member

marscher commented Mar 1, 2017

But probably I should move this test to a separate unit and provide a tiny test file

@marscher
Copy link
Member

marscher commented Mar 1, 2017

The failure is unrelated. Merging.

@marscher marscher merged commit c9ac876 into devel Mar 1, 2017
@marscher marscher deleted the fix_tica_all_constant branch March 1, 2017 18:16
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.

TICA error message when only 0's are passed
2 participants