View Issue Details

IDProjectCategoryView StatusLast Update
0015842MMW 5Extensions frameworkpublic2023-10-06 14:09
Reporterlowlander Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version5.0 
Target Version5.1Fixed in Version5.1 
Summary0015842: Plays from DLNA Server not scrobbled
DescriptionWhen MediaMonkey serves files to DLNA/Chromecast clients the plays aren't scrobbled to last.fm. With MM4 MediaMonkey does scrobble DLNA playback.
Additional Informationhttps://www.mediamonkey.com/forum/viewtopic.php?f=32&t=99113
TagsNo tags attached.
Attached Files
2186 last.fm UPnP.LOG (624,314 bytes)
Fixed in build2805

Relationships

related to 0007837 closedLudek MMW v4 DLNA/UPnP: Play counter increases after a few seconds of playback 
related to 0010454 closedLudek MMW v4 Playcount is not increased for auto-converted tracks sometimes 
related to 0008615 closedLudek MMW v4 Option to disable DLNA playcounts 
related to 0014043 assignedmartin MMA DLNA Playback: Playing using DLNA from MMW double scrobble 
related to 0008154 closedpeke Last.fm plugin Ability to disable scrobbling for DLNA devices 
related to 0016291 newrusty MMW 5 Last.fm Scrobbler: support for radio streams and non-library files 
related to 0006597 closedmichal MMW 5 Integrate scrobbler into MM 
related to 0016713 closedmichal MMW 5 Play to devices other than the internal player fails to scrobble 
related to 0020261 closedLudek MMW 5 Scrobble to last.fm doesn't remember setting 

Activities

michal

2019-07-29 11:40

developer   ~0054226

Rusty, please revise this request. I think, this is logically wrong - last FM script is bound to the MM5 player, it sends nowplaying info and scrobbles to lastFM based on MM5 player events. MM5 DLNA server can serve data to more clients, every client could have different lastFM account, some could use scrobbler, some not. I think this should not be feature of the server at all.

Ludek

2019-07-29 13:42

developer   ~0054229

Last edited: 2019-07-29 13:43

Note: If we decide to add DLNA scrobbling to MM5 too then it makes sense to make it customizable per client (like auto-conversions and shared content) already is. And probably also play counter and scrobbling should have separate checkbox? Also, the default for MM* family clients should be OFF.

rusty

2019-07-29 18:47

administrator   ~0054232

Last edited: 2019-07-29 18:49

I can understand both perspectives. i.e.
- Last.fm shouldn't be updated by the server because it's really something that should be managed by each client (different users/accounts; accuracy issues to to caching of partially played tracks)
- Last.fm should be updated by the server for remote devices used by the same user that the PC belongs to in the same manner that Playcounts can be updated for all remote clients (which they currently are by default). In fact the current default settings seem a bit strange that they track UPnP Play Counts (by default) but not last.fm scrobbles.

If we do implement this, as Ludek's suggested it probably also has ramifications for playcount handling over UPnP (0008615), since that currently doesn't support per-client rules, and it would make sense to implement both sets of functionality similarly. BTW, I'm not sure that it makes sense to have separate rules for play counts and scrobbling--It seems to me that a common approach could be used for both, especially since it wouldn't necessarily make sense to have scrobbling config in the MediaSharing dialog.

Possible approaches:

1) Simplest: Just modify the current implementation to support last.fm (in addition to UPnP). e.g.
Update play counter for items played over UPnP/DLNA --> Update database / play counts for items played over UPnP (this more generic wording applies better to last.fm)

2) Also modify the current implementation with custom rules. e.g. on the dialog

Media Sharing > DeviceName
-------------------------------------------------------------------
[UPnP/DLNA]
[ ] Customize for this device

[ ] Update database / play counts for items played over UPnP

rusty

2019-07-30 18:27

administrator   ~0054243

Last edited: 2019-07-30 19:08

3) The downside of 1) and 2) as pointed out by Martin is that if server UPnP playcounts/scrobbling is enabled, then remote clients playing this content may double-scrobble this. If 0014043 is implemented (it will give MMA the option to scrobble/or not content played from a remote server) then double scrobbles will be prevented, however, if the user prefers client-based scrobbling for server tracks, then Playcounts of the server-based tracks won't be updated.

So to cover that scenario, an alternative approach would be:
- Retain the existing wording of Update play counter for items played over UPnP/DLNA
- Add an option to [ ] Use last.fm (if enabled) to submit tracks played by UPnP clients
- Add the custom per-device rule for play counts

My preference is not to go with this third option because it probably only affects a small number of users, it's overly complex (last.fm UI must be split across two locations), and wording is necessarily clumsy. Note, though, that if 0014043 proves to not be technically feasible (and MMA is unable to disable scrobbling for playback of remote tracks), then we'll probably have to implement 3) since _all_ MMA clients that scrobble will likely be in a position where content will be double-scrobbled.

lowlander

2020-06-12 17:50

developer   ~0058523

Note that MediaMonkey also doesn't submit Chromecast plays and for Chromecast clients don't scrobble at all.

lowlander

2020-06-13 17:40

developer   ~0058543

To clarify, MediaMonkey 5 doesn't scrobble when Casting to DLNA and Chromecast Clients as well as not scrobbling when a Client plays from the Media Monkey Server (which MediaMonkey 4 did).

rusty

2020-06-14 03:54

administrator   ~0058546

OK, thanks. I've moved the 'Play to' issues to 0016713 since existing MM4 users (and any user) would expect that to work (unlike Serving content from MM5 in which the expectations are unclear and where the feature is technically problematic (since the server can't distinguish between plays and skips).

lowlander

2020-07-02 19:50

developer   ~0058738

Last edited: 2020-07-02 19:52

That's why an option in the Media Server settings to scrobble play can solve this. It's up to the user to enable and it can come with a warning that scrobbles could included skipped tracks and double scrobbles if client scrobbles too. If MMA has an option to exclude DLNA play from scrobbling we would have resolved it from a MediaMonkey environment.

An even better solution is to allow per client settings to override global play counter/scrobble settings so users can choose which clients count plays and/or scrobble.

Most hardware clients have no scrobbling capability, it's only in some App/Software clients that the client can scrobble and with Play To now scrobbling it will also be perceived as inconsistent by users that one method of DLNA playback scrobbles and the other doesn't.

lowlander

2021-04-12 16:27

developer   ~0062789

Note that MM5 also can't scrobble as a DLNA client (so neither MM5 Server nor MM5 Client can scrobble) as MM5 doesn't scrobble non-library files.

Ludek

2022-05-19 23:34

developer   ~0068183

OK, per client config for scrobbling / playcounter (for the clients that cannot scrobble) makes sense IMO.

Probably by introducint new "Playback rules" tab per client device:

Assigned to Rusty for the wording review / suggestions.
image.png (28,813 bytes)   
image.png (28,813 bytes)   

rusty

2022-09-16 02:23

administrator   ~0069311

Ludek's suggestion makes the most sense, BUT I doubt that there will be more than a handful of users taking advantage of this. So my vote would be to limit the effort on this and just make a single config entry.

e.g.

[x] Update play count when playing remotely
. [ ] Scrobble to last.fm if Scrobbling is enabled

Question: if we implement this, should we also implement a similar option for Synchronization e.g.
- on the last.fm panel:
. [ ] Include plays by synced devices

OR on each device profile:
. [ ] Scrobble to last.fm if Scrobbling is enabled

Ludek

2023-05-04 19:06

developer   ~0071731

Last edited: 2023-05-04 19:09

1) Implemented the UI, assigned to Michal to implement the scrobbling functionality.. Details discussed via IM..
2) Options > last.fm throws error in 5.1 builds

michal

2023-05-05 14:55

developer   ~0071742

Regression 2) fixed in build 2804.

michal

2023-05-09 18:06

developer   ~0071782

Fixed in build 2805.

peke

2023-05-19 22:01

developer   ~0071994

Last edited: 2023-05-19 22:06

I find it bit confusing that user needs to know to enable Last.FM in Server settings in order to scrobble remote plays.
eg. Now we have:
"Library -> Last.FM" settings and "Library -> Media Sharing -> Server settings -> Clients Tab -> Increase play count for remote plays + Scrobble to last.FM"

Otherwise both bugs are Verified in 2805

Edit: I personally think that introducing new playback rules tab (0015842:0068183) would be good for various reasons.
1. It would be per client setting that users can customize if needed
2. We can use that same tab for future fixes and customization eg. like disabling/customizing playback (fade, gap), custom Metadata sending, ...

rusty

2023-05-19 23:41

administrator   ~0072005

I agree that some users might expect to see this functionality in the last.fm settings. But given the limited number of users that will be taking advantage of this, I wouldn't spend any more time on this until we see strong demand for it.

It's probably a niche feature though, given that:
- The segment of our users that use both Last.fm and UPnP is probably very small
- Track counts over UPnP are not that accurate
- Track counts may include other users that the library owner might not want to count

peke

2023-05-19 23:47

developer   ~0072007

Agreed Closing

We can review this along 0016291 and 0014043