Skip to content

Strengthen out-of-season holiday filtering in discover recommendations - #104

Merged
dannyvfilms merged 1 commit into
latestfrom
claude/fix-seasonal-recommendations-z6c2u
Mar 6, 2026
Merged

Strengthen out-of-season holiday filtering in discover recommendations#104
dannyvfilms merged 1 commit into
latestfrom
claude/fix-seasonal-recommendations-z6c2u

Conversation

@dannyvfilms

Copy link
Copy Markdown
Owner

The previous out-of-season penalty (-0.14) was too weak to suppress Christmas movies (e.g. The Grinch, Jack Frost) in the "Top Picks For You" and "Comfort Rewatches" rows during non-holiday months. A highly-rated Christmas movie could score ~0.46 even after the penalty, still ranking in the top results.

Two changes:

  • Increase the out-of-season penalty from -0.14 to -0.40, which drops a full-strength holiday item from ~0.60 to ~0.20, pushing it well below typical non-holiday candidates.
  • Expand HOLIDAY_STRONG_TERMS to include "grinch", "krampus", and "nutcracker", and HOLIDAY_SOFT_TERMS to include "jack frost" and "santa claus", so iconic Christmas characters in titles are caught even when TMDB keyword tags are absent.

https://claude.ai/code/session_01XpYeZusWXu9yNUZiZMqzVw

Summary

  • Describe what changed and why.

Validation

  • List commands run and outcomes.

Migration Sync Gate (Required for dev -> latest sync PRs)

  • Conflicts resolved with upstream files preserved and fork behavior merged intentionally.
  • Migration conflicts handled per policy (no rewrite of shared/released migrations).
  • cd src && python manage.py makemigrations --merge run for affected apps.
  • cd src && python manage.py check_migration_hygiene --strict passed.
  • scripts/replay_upgrade_matrix.sh --from-tag <previous_release_tag> --to-ref latest --db sqlite,postgres --with-drift-scenarios passed.
  • coverage run src/manage.py test app users integrations lists events --parallel passed.

Notes

  • Link relevant issues (for example: Refs #101).

The previous out-of-season penalty (-0.14) was too weak to suppress
Christmas movies (e.g. The Grinch, Jack Frost) in the "Top Picks For You"
and "Comfort Rewatches" rows during non-holiday months. A highly-rated
Christmas movie could score ~0.46 even after the penalty, still ranking
in the top results.

Two changes:
- Increase the out-of-season penalty from -0.14 to -0.40, which drops
  a full-strength holiday item from ~0.60 to ~0.20, pushing it well
  below typical non-holiday candidates.
- Expand HOLIDAY_STRONG_TERMS to include "grinch", "krampus", and
  "nutcracker", and HOLIDAY_SOFT_TERMS to include "jack frost" and
  "santa claus", so iconic Christmas characters in titles are caught
  even when TMDB keyword tags are absent.

https://claude.ai/code/session_01XpYeZusWXu9yNUZiZMqzVw
@dannyvfilms
dannyvfilms merged commit 16af2a4 into latest Mar 6, 2026
1 of 2 checks passed
@dannyvfilms
dannyvfilms deleted the claude/fix-seasonal-recommendations-z6c2u branch March 27, 2026 00:00
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

Successfully merging this pull request may close these issues.

2 participants