search

gregversteeg / CorEx

CorEx or "Correlation Explanation" discovers a hierarchy of informative latent factors. This reference implementation has been superseded by other versions below.
GNU General Public License v2.0
303 stars 54 forks source link

Many 'print' statement in the code raise SyntaxError #3

Closed sbagri closed 8 years ago

sbagri commented 8 years ago

One of the failing message, for line 264 of corex.py is copied below, I am using python 3.5.1

print "Warning: Data matrix values should be consecutive integers starting with 0,1,..." ^ SyntaxError: Missing parentheses in call to 'print'

gregversteeg commented 8 years ago

Oh, sorry, I wrote everything for python 2.7. I really need to go back and make everything python3 compatible. I'll try to get to it eventually, but I'll also happily merge in these edits if someone else beats me to it.

sbagri commented 8 years ago

Added fix with pull request #4