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.

Sign up for GitHub

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

Jump to bottom

Minor issue in "lib/utils.py". #52

Open
u42dani opened this issue Mar 26, 2020 · 1 comment
Open

Minor issue in "lib/utils.py". #52

u42dani opened this issue Mar 26, 2020 · 1 comment

Comments

@u42dani
Copy link

u42dani commented Mar 26, 2020 •
edited by mdeff

I noticed line 139 in "lib/utils.py" should be modified
from model = gensim.models.Word2Vec(Sentences(self.documents), size)
to model = gensim.models.Word2Vec(sentences=Sentences(self.documents), size=size)
I guess the parameter list of the Word2Vec constructor changed...
@mdeff
Copy link
Owner

mdeff commented Jul 20, 2020

Thanks! Does that work with gensim==2.1.0 (the version used during the development of this code)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdeff @u42dani