this post was submitted on 29 Jan 2025
3 points (80.0% liked)
News Summary
116 readers
276 users here now
News feed with AI summary.
Rules:
- follow instance rules
- Engage in good faith discussion
- Tag relevant posts with [Meta] or [Request]
- meta = Discussion about the bot or AI summary.
- request = Development request or an rss feed to incorporate into the feed
- No calls for actionalable violence
- Congratulations your behavior is now a new rule
founded 1 month ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Looks like it counts the text "AI Summary" in the word count. The article is so short it dropped one unnecessary word, resulting in one more word total.
Holy shit that is the most edge case of edge cases lol. I don't think it counts the text "AI Summary" in the word count I assume its an issue with me going
len(text.split())is good enough. Guess I'm over counting somehow. I would be greatfull if someone could make a pull fixing this.Looking at the code, my guess was incorrect.
When counting the length of an article, it is not calculated correctly. It appears that this is because spaces before periods aren't stripped out before performing the count, making those periods count as their own words.
I'm too tired to figure out how to make it let me push to a branch right now. I have opened an issue that explains how to fix it.
Thx. FYI, U do it by making a fork of the project that then u can make a merge request from ur fork back to the original via the web ui.