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

Document how to get complete document distribution from LDA #2489

Open
Antupis opened this issue May 10, 2019 · 5 comments · May be fixed by #2625
Open

Document how to get complete document distribution from LDA #2489

Antupis opened this issue May 10, 2019 · 5 comments · May be fixed by #2625
Labels
difficulty easy Easy issue: required small fix documentation Current issue related to documentation good first issue Issue for new contributors (not required gensim understanding + very simple) Hacktoberfest Issues marked for hacktoberfest

Comments

@Antupis
Copy link

Antupis commented May 10, 2019

Document how to to get full document distribution from LDA, that would help in those cases where you are implementing your own distance algorithm and you need full distribution. Personally I set minimum_probability=0.0 but there might be other ways.
@piskvorky
Copy link
Owner

piskvorky commented May 10, 2019 •
edited

Yes, this is a common request. We should document the minimum_probability=0.0 + matutils.sparse2full() for a dense numpy conversion more prominently.

@Antupis can you open a PR with your suggested solution (updated docs)?

@piskvorky piskvorky added documentation Current issue related to documentation difficulty easy Easy issue: required small fix labels May 10, 2019
Antupis added a commit to Antupis/gensim that referenced this issue May 11, 2019
@Antupis
Documentation changes for piskvorky#2489
85a31e9
@Antupis Antupis mentioned this issue May 11, 2019
Closed
@mpenkov mpenkov added good first issue Issue for new contributors (not required gensim understanding + very simple) Hacktoberfest Issues marked for hacktoberfest labels Sep 28, 2019
@Hiyorimi
Copy link
Contributor

Hiyorimi commented Oct 5, 2019

Can I finish this one?

@mpenkov
Copy link
Collaborator

mpenkov commented Oct 6, 2019

@Hiyorimi Sure, please go ahead.

Hiyorimi added a commit to Hiyorimi/gensim that referenced this issue Oct 7, 2019
@Hiyorimi
Documentation changes for piskvorky#2489 piskvorky#2491
b43a301
@Hiyorimi Hiyorimi linked a pull request Oct 7, 2019 that will close this issue
Open
@Hiyorimi
Copy link
Contributor

Hiyorimi commented Oct 7, 2019

@mpenkov take a look please. I didn't get whole BoW part? What do should I describe in this example?

@Hiyorimi
Copy link
Contributor

Hiyorimi commented Oct 8, 2019

@piskvorky Can you take a look please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty easy Easy issue: required small fix documentation Current issue related to documentation good first issue Issue for new contributors (not required gensim understanding + very simple) Hacktoberfest Issues marked for hacktoberfest
Projects
None yet
Milestone
No milestone
4 participants
@piskvorky @mpenkov @Hiyorimi @Antupis