Radim Řehůřek

.

Thanks. Can you add some benchmarks into the PR description? Something to point people to, a TL;DR for the release notes. Also a link to this "POT" dependency, its license...

Thanks for reporting. @gojomo any ideas? IIRC you rewrote this part for Gensim 4.0.

Yes, the test data could use a clean up. There are open tickets around that such as #2967. But honestly low priority, so I have no idea when we'll get...

Thanks for the investigation @gojomo. That matches what I remember – a non-issue except for highly theoretical what-if scenarios. Which, while valid, are zero priority for me right now. But...

Yes, this limit is arbitrary and feels weird. I'd also prefer to either get rid of it or be more explicit about its existence. I wouldn't phrase this as a...

Only to the degree I trust [my memory](https://github.com/RaRe-Technologies/gensim/commit/6c8f478811cad9ea4138de3594bf355b610eb5f7) (which is to say, not much). Getting rid of the limit altogether would definitely result in a more beautiful algorithm, no matter...

Thanks! Yeah I don't think those files are needed for anything, once Gensim has been installed. A fix should be as easy as excluding those files in MANIFEST, right? Or...

@mpenkov WDYT? Not urgent (low impact) so removing the Next Release flag. But if trivial, we might as well clean up these files too.

Good question – what's the motivation here @pabs3 ? Freeing up disk space? Or do these files make some local "modify-and-recompile" workflow harder?