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

Fix Doc2Vec.infer_vector after loading old Doc2Vec (gensim<=3.2). Fix #1952 #1974

Merged
merged 3 commits into from
Mar 12, 2018

Conversation

manneshiva
Copy link
Contributor

Fixes #1952.

@manneshiva manneshiva changed the title Fixes inferring vector for new document on loading old Gensim (<= version 3.2) model Fix inferring vector for new document after loading old Gensim (<= version 3.2) model Mar 11, 2018
@manneshiva manneshiva changed the title Fix inferring vector for new document after loading old Gensim (<= version 3.2) model Fix inferring vector for new document after loading old Doc2Vec (Gensim <= version 3.2) model Mar 11, 2018
@menshikh-iv menshikh-iv changed the title Fix inferring vector for new document after loading old Doc2Vec (Gensim <= version 3.2) model Fix Doc2Vec.infer_vector after loading old Doc2Vec (gensim<=3.2). Fix #1952 Mar 12, 2018
@menshikh-iv menshikh-iv merged commit 122dad6 into piskvorky:develop Mar 12, 2018
thatandromeda added a commit to thatandromeda/hamlet that referenced this pull request Oct 15, 2021
I can't get the fast C binding stuff working on Amazon Linux, so the
features involving processing uploaded text fail (they rely on a code
path that only exists with the fast C backing). I'd still like to get
it working for the sake of speed, but in the meantime, I'd like the
feature to work at all.

I tried the next minor version of gensim (3.3.0) but it failed due to
the bug fixed in this PR, which was incorporated into 3.5.0:
piskvorky/gensim#1974

So I'm using the lowest gensim version which passes the tests. It's way
out of date now but I don't want to open up the upgrade can of worms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants