-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
per_word_topics for a corpus #908
Comments
@tmylk is this issue still open. |
@saksham15082 It is still open and a fix would be appreciated. We need to pass all the params including per_word_topics through the apply call. |
@tmylk The link to "mailing list" redirects elsewhere, for a better understanding of this issue. Please fix. thanks. |
@markroxor Thanks for pointing it out. Link updated to https://groups.google.com/d/msg/gensim/RgiN22vtPKU/3K3TpxjKBgAJ |
@markroxor please go ahead with this issue. |
Fixed in #978 Just needs better tests |
As reported on the mailing list
lda.get_document_topics(clipped_corpus, per_word_topics=True)
doesn't return per_word_topics.This line should be changed to pass all the params including
per_word_topics
through theapply
call.The text was updated successfully, but these errors were encountered: