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

Reading speed wpm sometimes to high #1373

Closed
AngelCrawford opened this issue Oct 12, 2022 · 3 comments
Closed

Reading speed wpm sometimes to high #1373

AngelCrawford opened this issue Oct 12, 2022 · 3 comments
Labels

Comments

@AngelCrawford
Copy link

AngelCrawford commented Oct 12, 2022

I updated right now to the new 2.19 version, and added the new reading_speed_wpm to my config file.

On some articles the reading speed wpm is really high, and on other articles it works just fine.
I can't say why some articles this long, could it be the reviews under the post, that will add words to the count? Or the adds on those sides?

image
image

PS: Thanks for the lovely tool! I really ❤️ it!

@jtojnar jtojnar added the bug label Oct 12, 2022
@jtojnar
Copy link
Member

jtojnar commented Oct 12, 2022

Thanks for reporting. Are you using regular RSS spout or the Content extraction one? In both cases selfoss reports for me that the first article has 1248 words:

HTTP response containing word count in developer tools

I was not able to reproduce with reading_speed_wpm=200:

Reasonable reading time detected

@AngelCrawford
Copy link
Author

AngelCrawford commented Oct 12, 2022

You probably will hate me xD I found the issue. I added a 1 after the reading_speed_wpm to activate it, but you need to add the words per minute you are able to read so a 200 for instance... it's late where I live >.<
Thanks for checking!

@jtojnar jtojnar added question and removed bug labels Oct 12, 2022
@jtojnar
Copy link
Member

jtojnar commented Oct 12, 2022

Heh, I guessed something like that might be the case but discarded the hypothesis when I thought your reading time was greater than the number of words, which would mean wpm < 1, which would just be trimmed to 0, so that could not be happening. But I actually misread the number in your screenshot 🤦‍♀️ here too.

@jtojnar jtojnar closed this as completed Oct 12, 2022
jtojnar added a commit that referenced this issue Oct 13, 2022
These lead to confusion like:

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

No branches or pull requests

2 participants