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.

Sign up for GitHub

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

Jump to bottom

Update show_topics method from Mallet wrapper #1747

Closed
menshikh-iv opened this issue Nov 30, 2017 · 0 comments
Closed

Update show_topics method from Mallet wrapper #1747

menshikh-iv opened this issue Nov 30, 2017 · 0 comments
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix good first issue Issue for new contributors (not required gensim understanding + very simple)

Comments

@menshikh-iv
Copy link
Contributor

Need to update https://github.com/RaRe-Technologies/gensim/blob/3.1.0/gensim/models/wrappers/ldamallet.py#L250 (replace num_words to topn) to prevent warning (report comes from https://groups.google.com/forum/#!topic/gensim/r3YIaqPRmzU)
@menshikh-iv menshikh-iv added bug Issue described a bug difficulty easy Easy issue: required small fix good first issue Issue for new contributors (not required gensim understanding + very simple) labels Nov 30, 2017
@apoorvaeternity apoorvaeternity mentioned this issue Dec 1, 2017
Merged
menshikh-iv pushed a commit that referenced this issue Dec 5, 2017
@apoorvaeternity @menshikh-iv
Replace num_words to topn in LdaMallet.show_topics. Fix #1747 (#1749)
aad849a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix good first issue Issue for new contributors (not required gensim understanding + very simple)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@menshikh-iv