Radim Řehůřek

Results 318 comments of Radim Řehůřek
trafficstars

@isamaru what's the status here? Are we raising NotImplementedError consistently? If not, can you list what needs to be done, so others can take this over?

Thanks @alreadytaikeune . The training loss is a recent addition and may contain errors. What is your use-case for using it? How did you find about this issue? Re. your...

Thanks! Can you run the benchmark too? Simply train a model on the [text9 corpus](https://markroxor.github.io/gensim/static/notebooks/Word2Vec_FastText_Comparison.html) with/without this PR, using 4 parallel workers and computing / not computing the loss. Then...

Thanks a lot @alreadytaikeune ! That sounds good to me. Let's wait for @menshikh-iv 's final verdict & merge, once he gets back from holiday.

@gojomo is right. I'm not aware of any changes to the CLI version of word2vec in Gensim. IIRC, the command line version was created to match the CLI of the...

Thanks! I'm also not sure what the "canonical" vs generated paths are, or Sphinx `*.py` duplication, deferring to @mpenkov .

I don't see any `int()` in your example either – how do you mean?

@TLouf can you fix the conflict please? After that we should be ready to merge @mpenkov – unless you see something off!

I think it's fair to say this PR is stale / dead. Unless someone picks it up to push it over the line, it's not happening, sorry.

We want the patch to go the the latest (`develop`), yes. Thank you!