Skip to content
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

Update requirements to prevent gensim breaking #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tyronechen
Copy link

Dear authors, thank you for the useful package. With updates to gensim, scipy and scikit-learn, the old requirements.txt file no longer works. This pull request resolves that.

In the original requirements.txt file, gensim >=3.4 was specified. After the release of gensim >=4, API changes prevented code from running. The new requirements file fixes gensim at 3.4.

Errors were obtained when running with some versions of scikit-learn and scipy. scikit-learn==0.18.2 and scipy==1.2.0 were tested and found to work.

MulticoreTSNE==0.1 was also added to requirements file.

Finally, existing installs of certifi may cause problems. This can be fixed by specifying --ignore-install certifi to pip install. README.md was updated with this note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant