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.

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

summarize(text) should give minimum 1 word #47

Open
lukakostic opened this issue Sep 16, 2018 · 2 comments
Open

summarize(text) should give minimum 1 word #47

lukakostic opened this issue Sep 16, 2018 · 2 comments

Comments

@lukakostic
Copy link

summarize(text) shouldnt give empty strings. It should retry summarizing with a greater rate until it achieves at least some text.

Also with this input:

"
Hi, I’ve got 2 designs for my landing page, one for Mobile, one for Desktop. There is no animation. I want the website to be done in the coming days. This job was posted from a mobile device, so please pardon any typos or any missing details.
"

It says only keyword is "got" which doesnt seem right.

@FeiGSSS
Copy link

FeiGSSS commented Sep 30, 2018

I encountered this kind of problem when I'm trying to summarize the following paragraph:

ALBUQUERQUE, N.M. (AP) — The Latest on Martin Luther King Jr. Day celebrations around the country (all times local):4:15 p.m.Former Vice President Joe Biden says American values are being challenged in present times but that the leadership of Martin Luther King Jr. provides an example of how to respond.Biden was the keynote speaker Monday at the Delaware State Bar Association's annual Martin Luther King Jr. Day breakfast in Wilmington, Delaware.The News Journal reports Biden criticized President Donald Trump's response to white nationalist rallies last August in Charlottesville and his reported use of an obscenity to describe African nations and Haiti during a recent immigration discussion with some Congressional leaders.But he said he believes Americans are ready to respond and re-establish the nation's "moral fabric."Said Biden, now is the time to "remind ourselves who we are as Americans."____3:15 p.m.With Vice President Mike Pence sitting in the pews, a Maryland pastor denounced President Donald Trump's vulgar description of African nations.Maurice Watson, pastor of Metropolitan Baptist Church in Largo, delivered the sermon Sunday while Pence was visiting.Watson called Trump's remarks — in which he reportedly used an obscenity to describe African nations and Haiti during an immigration discussion with Congressional leaders — "dehumanizing" and "ugly."The pastor said "whoever made such a statement" is wrong and should be held accountable.Watson said he felt "led by God to do it" and noted many of his congregants come from African nations.Worshippers stood and applauded as Watson spoke.WUSA-TV reports Pence became red-faced at times during the sermon, though Pence's office denied that in an email Monday to The Associated Press.

default setting were used:
summarizer.summarize(text)
but it returns a empty string

@hiyamgh
Copy link

hiyamgh commented Sep 29, 2020

So there is no solution to this problem ? I am also getting an empty string

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants