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 gensim.fasttext usage #754

Open
irinakhismatullina opened this issue Apr 11, 2019 · 2 comments
Open

Update gensim.fasttext usage #754

irinakhismatullina opened this issue Apr 11, 2019 · 2 comments
Assignees
Labels

Comments

@irinakhismatullina
Copy link
Contributor

New release of gensim==3.7.2 broke several things in CandidateGenerator in typos:

  1. Loading fasttext model from the file - easy fix.
  2. Modelforge save/load functions are now broken because of the changed attributes in FasttextKeyedVectors class - not nice, need to rewrite the _generate_tree/_load_tree, but doable.
  3. Because of (2) new models for usage and testing should be trained, and that's the worst of all (reports inconsistencies and so on).

So due to (3) it's suggested to leave it for the future releases, when the new models will be integrated anyway.

@zurk
Copy link
Contributor

zurk commented May 23, 2019

@irinakhismatullina I think this one was done, wasn't it?

@irinakhismatullina
Copy link
Contributor Author

Nope, we just put upper boundary on the gensim version.

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

No branches or pull requests

2 participants