style-analyzer icon indicating copy to clipboard operation
style-analyzer copied to clipboard

Update gensim.fasttext usage

Open irinakhismatullina opened this issue 6 years ago • 2 comments

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.

irinakhismatullina avatar Apr 11 '19 10:04 irinakhismatullina

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

zurk avatar May 23 '19 12:05 zurk

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

irinakhismatullina avatar May 23 '19 12:05 irinakhismatullina