Repository navigation
Music moods and styles from Plex, and the player of the night in the Wrap-Up - #242
Merged
Merged
Conversation
…Wrap-Up Nothing said how the music felt. A custodian walk reads Plex's own moods and styles per artist, album and track into music_tags, through Plex's filter lists; a play takes its track's moods, else its album's, else its artist's, found by key or, for history events without a ratingKey, by artist and album names. The Wrap-Up reads the leading moods and styles, the mood of each part of the day, and the players by hours from watch_history.player, with two new cards (FACTS_VERSION 10). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KECJpoVqo39hQKxkrxUjrc
Randomname653
force-pushed
the
claude/bold-johnson-w1warc
branch
from
October 25, 2026 11:49
7c0f2e1 to
f0803b1
Compare
Owner
Author
|
Generated by Claude Code |
# Conflicts: # src/services/data_custodian.py # src/services/wrapup_facts.py
…e it 10) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
This PR adds the Plex music moods and styles from telemetry priority 6 in
docs/WRAPUP_V2_BLUEPRINT.md. Track genres were a comma string, so nothing told the Wrap-Up how the music felt. It could not say "mornings ran Energetic, nights Melancholy" or "after 23:00 it was the iPhone".Rebuilt on current
main: the player part is gone, and the player now comes from thewatch_history.player/platformthat #241 added.Moods and styles (
services/music_tags.py, tablemusic_tags)music_tagsruns every 72 h without a model. It reads Plex's own moods and styles for every music section, the way Plex's filter bar does./library/sections/{key}/{mood|style}?type=…), then the items under each tag.Mood/Styleare read too.track_lyrics) by the play's key.Wrap-Up (
FACTS_VERSION10; 9 is taken by #243)music.moods: the leading moods and styles over the played time, and the leading mood of each part of the day on the user's clock. It needs at least a third of the music time tagged.devices: the players by hours (player, elseplatformread as its family, so "iOS 17.1 (iPhone14,2)" counts as iPhone) and the player of the night. It needs at least half the time with a known player, and at least two players.When this is merged, the
artist_tagsjob from #241 can go (the owner takes it out).Checks
python tests/run_all.pyis green (256/256 suites)tests/test_music_moods.py(25 checks) covers:main🤖 Generated with Claude Code
https://claude.ai/code/session_01KECJpoVqo39hQKxkrxUjrc