Skip to content

fix: include youtube in freshness calculation (#108)#112

Open
sjhddh wants to merge 1 commit intomvanhorn:mainfrom
sjhddh:fix/issue-108
Open

fix: include youtube in freshness calculation (#108)#112
sjhddh wants to merge 1 commit intomvanhorn:mainfrom
sjhddh:fix/issue-108

Conversation

@sjhddh
Copy link
Copy Markdown

@sjhddh sjhddh commented Mar 26, 2026

Summary

Fixes #108 by including YouTube items in the data freshness evaluation.

Changes

  • Added yt_recent to calculate recent YouTube videos
  • Included yt_recent in total_recent sum
  • Included len(report.youtube) in total_items sum

Fixes #108
@mvanhorn
Copy link
Copy Markdown
Owner

mvanhorn commented Mar 28, 2026

Thanks for tackling #108. Including YouTube in the freshness calculation is the right call. The v3.0 release rewrites the freshness and scoring pipeline, so I can't commit to merging this as-is. I'll make sure this gets looked at once 3.0 is in.
@sjhddh
Copy link
Copy Markdown
Author

sjhddh commented Mar 31, 2026

Sounds great! Looking forward to the v3.0 release. Let me know if you need any help adapting this for the new scoring pipeline once it is out.
@mvanhorn
Copy link
Copy Markdown
Owner

mvanhorn commented Mar 31, 2026

Thx much
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@sjhddh @mvanhorn