evolaemp / online_cognacy_ident

Fast and unsupervised methods for multilingual cognate clustering (Rama, Wahle, Sofroniev, and Jäger)
MIT License
1 stars 1 forks source link

Python Version #3

Closed erathorn closed 6 years ago

erathorn commented 6 years ago

We should go for python 3.5 or so. Currently, an installation of python-igraph under any version of windows is not possible for python 3.6.

LinguList commented 6 years ago

somehow, I ran into an igraph issue here on my 3.5 version, so I had to resort to UPGMA. No idea why. But there may be some update to igraph, which would also mean that changing the igraph-cluster code is in order.

pavelsof commented 6 years ago

@erathorn, did the binaries from the link you added to the README fix the problem for 3.6? I guess that providing the helpful link is as much as we can do unless we can replace the package with another one.

@LinguList, I remember having issues installing igraph on my last laptop running Ubuntu 14.04 (but I do not have problems now on 16.04).. but unfortunately I do not remember how I fixed them back then (if I did at all).

erathorn commented 6 years ago

The last time I checked, there were no igraph binaries for python 3.6. This was the main reason for me to suggest using python 3.5. If I remember correctly, there were also some problems for other dependencies with python 3.6.

PhyloStar commented 6 years ago

I am using 3.5 with igraph on a new laptop since yesterday. All is well as of now. :)

pavelsof commented 6 years ago

What about adding this clarification at the very end of the windows users paragraph: (but even those might not work with Python 3.6 and you might need to use Python 3.5 instead).

The alternatives would be to use another package or implement the InfoMap algorithm ourselves.

LinguList commented 6 years ago

my bug is resolved now. It was my stupidity, as we have tests not passed on lingpy via travis, as they'd require too many dependencies, so I only spotted it when really using it, but it's solved now.

erathorn commented 6 years ago

@LinguList So igraph on 3.5 works for you?

LinguList commented 6 years ago

yes, all works fine on ubuntu and archlinux.

erathorn commented 6 years ago

@pavelsof If there are no functionalities in the code which are available under 3.6 but not 3.5 we can stick to 3.6. Otherwise, I would suggest changing the Python version to 3.5

pavelsof commented 6 years ago

I am also using 3.5 on my machine just because this is the default on Ubuntu. However, if somebody has 3.6 (e.g. Mattis on his Arch Linux should have that by default) and they do not have issues installing igraph, we should not restrict them. In other words, we can say that we have been using Python 3.5 but we should not say that users must use 3.5 because chances are that the restriction would be unnecessary (e.g. if Mattis was to strictly follow our instructions, he would have to go through the hassle of manually building Python 3.5).

What about adding that sentence at the very beginning of the setup section: The code is developed using Python 3.5 but later versions should also do as long as the dependencies are satisfied. ...and leave the windows users paragraph as it is now?

erathorn commented 6 years ago

Good idea!

pavelsof commented 6 years ago

Done with 3f2e4587df3677a18001671e510bbe2e4f23b6ab :)

LinguList commented 6 years ago

What Robert usually is doing is saying: this code was tested in machine X with version Y, etc., and (and this is also important): if you use this code and run into difficulties, make sure you try and run it in a fresh virtual environment (this avoids people coming with personal bugs)

pavelsof commented 6 years ago

I am not sure about the former but I like the latter and added it with 67e52f4bd683eab0842fda006786e22d5d7def89 :)