Opened 12 years ago

Closed 8 years ago

#4708 closed patch (Fixed)

New "intelligent ordering"

Reported by: mythtv@… Owned by: paulh
Priority: minor Milestone: 0.25
Component: Plugin - MythMusic Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Current "intelligent ordering" seems to have side effects (these days I always get 4 stars whereas I have many 8 to 10 stars ratted songs).

Idea in this patch is to compute weights for each song, then compute a random order, balanced with the computed weights.

Note: this patch does not use the "IntelliRandomWeight?", so if it is accepted, the GUI and database need to be updated to remove this option.

Attachments (4)

mythmusic-intelligent-order.diff (7.2 KB) - added by mythtv@… 12 years ago.
mythmusic-intelligent-order-v2.diff (7.7 KB) - added by mythtv@… 12 years ago.
Normalized weights using the number of songs having the same rating
mythmusic-intelligent-order-trunk-19938.diff (7.8 KB) - added by mythtv@… 11 years ago.
Updated patch: apply cleanly on current trunk.
mythmusic-intelligent-order-trunk-22065.diff (7.7 KB) - added by mythtv@… 10 years ago.
Adapted patch to apply and compile on current trunk.

Download all attachments as: .zip

Change History (11)

Changed 12 years ago by mythtv@…

comment:1 Changed 12 years ago by stuartm

Milestone: unknown0.22

Changed 12 years ago by mythtv@…

Normalized weights using the number of songs having the same rating

Changed 11 years ago by mythtv@…

Updated patch: apply cleanly on current trunk.

comment:2 Changed 10 years ago by stuartm

Milestone: 0.220.23

Mythmusic will be getting a lot of attention for 0.23, I'll look at this then.

comment:3 Changed 10 years ago by stuartm

Owner: changed from Isaac Richards to stuartm
Status: newassigned

Changed 10 years ago by mythtv@…

Adapted patch to apply and compile on current trunk.

comment:4 Changed 10 years ago by stuartm

Component: mythmusicPlugin - MythMusic

comment:5 Changed 10 years ago by stuartm

Milestone: 0.23unknown
Owner: stuartm deleted
Status: assignednew

Dropping this one back into the pool for someone else to look at.

comment:6 Changed 10 years ago by stuartm

Owner: set to paulh
Status: newassigned
Version: 0.21-fixesTrunk Head

comment:7 Changed 8 years ago by paulh

Milestone: unknown0.25
Resolution: Fixed
Status: assignedclosed

Thanks for the patch. This has been added to current master. Please report back if you notice any problems.

Note: See TracTickets for help on using tickets.