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

ZeroDivisionError: float division by zero #2075

Closed
anujgupta82 opened this issue Jun 1, 2018 · 4 comments · Fixed by #2154
Closed

ZeroDivisionError: float division by zero #2075

anujgupta82 opened this issue Jun 1, 2018 · 4 comments · Fixed by #2154
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix

Comments

@anujgupta82
Copy link

anujgupta82 commented Jun 1, 2018 •
edited
Loading

Getting error : ZeroDivisionError: float division by zero

https://github.com/RaRe-Technologies/gensim/blob/9481915915bf61aa6e4e719a2f26d509677e6779/gensim/summarization/pagerank_weighted.py#L53

error
@menshikh-iv menshikh-iv added the need info Not enough information for reproduce an issue, need more info from author label Jul 30, 2018
@menshikh-iv
Copy link
Contributor

Hi @anujgupta82, please give us more information about this problem

  • OS version
  • Python version
  • Gensim version
  • Code example for reproducing this error (with all needed data)

@menshikh-iv
Copy link
Contributor

ping @anujgupta82

@lshostenko
Copy link
Contributor

#2154

@menshikh-iv menshikh-iv added bug Issue described a bug difficulty easy Easy issue: required small fix and removed need info Not enough information for reproduce an issue, need more info from author labels Aug 10, 2018
@menshikh-iv
Copy link
Contributor

Thank you @lshostenko

Just duplicate an example for reproducing an issue here

from gensim.summarization import keywords

keywords('gensim')  # raises ZeroDivisionError
keywords('sitio construcción. estaremos línea', deacc=False)  # raises ZeroDivisionError
keywords('sitio construcción. estaremos línea', deacc=True)  # returns empty result

@lshostenko lshostenko mentioned this issue Aug 10, 2018
Merged
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
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Return keywords before matrix processing if no graph edges. Fixes #2075 lshostenko/gensim
3 participants
@lshostenko @anujgupta82 @menshikh-iv