LdaSeqModel word probabilities (too rough rounding)

[menshikh-iv]

Problem: Probabilities of words don't sum up to 1 because we have round(_, 3) in print_topic method
Solution: Remove round from print_topic from here

Based on message from maillist

@bhargavvader what's a reason to use round in this place?

[myrthevandieijen]

@menshikh-iv and @bhargavvader, thanks so much for looking into this issue! Do you have any idea when this problem will be fixed? Many thanks in advance!

[menshikh-iv]

@myrthevandieijen It's very easy to fix, but I want to get an explanation from @bhargavvader about this behavior.

[bhargavvader]

I really can't seem to remember why i had used round over here - can anyone make the fix and then see if the tests still pass fine?

[myrthevandieijen]

Any progress on this issue so far?Thanks!

[menshikh-iv]

Fixed in #1547, @myrthevandieijen you can install gensim now from develop branch OR wait for next release (will be in September).

[myrthevandieijen]

Great! Thank you very much! 2017-08-24 12:05 GMT+02:00 Menshikh Ivan <notifications@github.com>:

…

Fixed in #1547 <#1547>, @myrthevandieijen <https://github.com/myrthevandieijen> you can install gensim now from develop branch OR wait for next release (will be in September). — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1480 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AJwblYe5xcJwSt0hi_enXEBoErT7YoaNks5sbUr4gaJpZM4OTunJ> .

--

______________________________________________________ *Myrthe van Dieijen - Agamemnonstraat 59-1 - 1076LS Amsterdam - +31 6 516 39 992*