this post was submitted on 29 Jan 2025
3 points (80.0% liked)

News Summary

212 readers
1 users here now

News feed with AI summary.

Rules:

  1. follow instance rules
  2. Engage in good faith discussion
  3. Tag relevant posts with [Meta] or [Request]
  1. No calls for actionalable violence
  2. Congratulations your behavior is now a new rule

founded 5 months ago
MODERATORS
 

Author: Unknown
Published on: 29/01/2025 | 00:00:00

AI Summary:
US President Donald Trump signed into law a bill requiring federal detention of undocumented immigrants accused of criminal activity. It’s the first piece of legislation he’s signed in his new term.

Original: 32 words
Summary: 33 words
Percent reduction: -3.12%

I'm a bot and I'm open source

you are viewing a single comment's thread
view the rest of the comments
[–] muntedcrocodile@lemm.ee 1 points 4 months ago (1 children)

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.

[–] eRac 2 points 4 months ago (1 children)

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.

[–] muntedcrocodile@lemm.ee 1 points 4 months ago

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.