Skip to content
This repository has been archived by the owner on May 21, 2022. It is now read-only.

Enhanced Submission Rules - don't scrobble podcasts / ebooks #127

Closed
konsept opened this issue Mar 30, 2014 · 20 comments
Closed

Enhanced Submission Rules - don't scrobble podcasts / ebooks #127

konsept opened this issue Mar 30, 2014 · 20 comments
Labels
bugs Functionality is broken. Ambiguous. feature request optional issues

Comments

@konsept
Copy link

konsept commented Mar 30, 2014

Is it possible to support additional submission criteria such as excluding Podcast plays from scrobbling?

@inverse
Copy link
Contributor

inverse commented Mar 30, 2014

How would you determine what a podcast play is?

@konsept
Copy link
Author

konsept commented Mar 30, 2014

Genre?

@inverse
Copy link
Contributor

inverse commented Mar 31, 2014

Hmm this could work from a little reading that I've done. See this quote:

Nearly all podcasts use the genre "Podcast", which is not even included in the official ID3 genre list.

I don't know if track genre is being served up though. I guess it could be a proposition to extend sls integration mechanism.

@konsept
Copy link
Author

konsept commented Apr 1, 2014

Excellent. Yes, I too have seen Podcast used as the genre in nearly all of those I subscribe to. MediaMonkey maintains a Type field that classifies files as Music, Podcast, Video, and the like but this may be app-specific so I'm unclear on whether this is contained within ID3.

@tgwizard tgwizard changed the title Enhanced Submission Rules Enhanced Submission Rules - don't scrobble podcasts / ebooks Jul 24, 2014
@tgwizard
Copy link
Collaborator

It isn't certain that SLS can read the genre data, or any other metadata except the track name, artist name, and album name. Perhaps also the duration.

Perhaps there is some other way to distinguish these kinds of tracks - duration > 30min? "[podcast]" in the name? Or something else.

It might be possible to configure in SLS to "dont scrobble tracks longer than X minutes/seconds". What do you think?

@inverse
Copy link
Contributor

inverse commented Jul 24, 2014

I guess having the ability to disable scrobbling for tracks that are longer then X would be a work around. Alternatively, in my opinion what feels better would be a blacklist filtering mechanism.

For example most podcasts I have seen come in a format similar to:

Laravel IO - Episode 15 - ActiveRecord, Laravel Haterade, DHH and Laracon Stuff with Taylor Otwell, Jeffrey Way, Mitchell van W

So perhaps allowing the user to specify some regular expressions of what to block. Which for the above would be something like ^Laravel IO - Episode \d+ - .+. What do you think of this?

@eligrey
Copy link

eligrey commented Jul 24, 2014

@tgwizard A bunch legitimate music and ambient tracks (that are not full albums or compilations) in my library are close to or over 30 minutes:

  • The original Rainy Mood track (ambient, 34m 23s)
  • Children In The City by A.M. Architect (music, 27m 54s)
  • When Can (flavors mix) by Kettel & Secede (music/ambient, 45m 52s)
  • 0181 by Four Tet (music, 38m 8s)
  • Perlence subrange 6-36 by Autechre (ambient, 58m 35s)

If all the other data you can get is just track name, artist name, and album name, then unfortunately the best solution seems to just be a manually curated app blacklist (I assume you can figure out which app is sending the now playing notification?).

@inverse
Copy link
Contributor

inverse commented Jul 24, 2014

How do you think the handling of of a blacklist list feature should be handled? I feel we have one of two options.

  1. Implement a global one
  2. On a per app basis - although I've not looked into the architecture of that part of the application yet so I don't know if would lend itself to that without refactoring.

Opinions?

@eligrey
Copy link

eligrey commented Jul 24, 2014

The per-app basis would be pretty complicated for users to use, as I would imagine you would have to combine that with the filtering as you suggested before. I wouldn't be against it, but it would definitely have to be put in some advanced settings menu.

I would prefer the global one due to simplicity. This will cover for-purpose apps like podcast apps and audiobook apps like Audible.

@konsept
Copy link
Author

konsept commented Jul 24, 2014

I agree with @eligrey as I also have tracks longer than 30min. I think the regular expression idea by @inverse could work well if reading genre data in SLS is not an option. Probably support for multiple exclusion criteria would be necessary. For example, exclude both *Podcast* and *Episode* or the name of the podcast provider such as BBC*. From my perspective, I don't see the need for this to be app-specific but if you see a need, by all means.

@eligrey
Copy link

eligrey commented Jul 24, 2014

Also, to add to your filter list if you guys do decide to have default filters: ^Chapter\s*\d+$ on the track name will usually cover Audible audiobooks.

@inverse
Copy link
Contributor

inverse commented Jul 24, 2014

Yeah - I think a global one would be a better fit for the app and would certainly complicate things less for the end user.

@tgwizard
Copy link
Collaborator

Regexps are fine for advanced users, but how do we explain this to regular non-regexp users? Perhaps just a list of "don't scrobble if track contains any of these words"? What about album, and artist?

@tgwizard
Copy link
Collaborator

If anyone wants to look at the code, they intents broadcasted by the media players (containing the track info) is received by BroadcastReceivers such as https://github.com/tgwizard/sls/blob/master/src/com/adam/aslfms/receiver/AndroidMusicReceiver.java. The logic to filter tracks could perhaps be placed here: https://github.com/tgwizard/sls/blob/master/src/com/adam/aslfms/receiver/AbstractPlayStatusReceiver.java#L99 (after we check if we should scrobble this music app).

@modu
Copy link

modu commented Jul 26, 2014

I thought of printing all the keys of bundle received from default android music player and a podcast app . Can anyone find any criteria that we can use from this list to filter normal music from a podcast?
Tried printing values of keys but app crashes
From default Android music player app( from log)

"
duration is a key in the bundle
playstate is a key in the bundle
currentContainerName is a key in the bundle
artist is a key in the bundle
domain is a key in the bundle
currentSongLoaded is a key in the bundle
preparing is a key in the bundle
rating is a key in the bundle
albumId is a key in the bundle
currentContainerTypeValue is a key in the bundle
currentContainerId is a key in the bundle
playing is a key in the bundle
streaming is a key in the bundle
inErrorState is a key in the bundle
albumArtFromService is a key in the bundle
id is a key in the bundle
currentContainerExtData is a key in the bundle
album is a key in the bundle
local is a key in the bundle
track is a key in the bundle
position is a key in the bundle
currentContainerExtId is a key in the bundle
supportsRating is a key in the bundle
ListSize is a key in the bundle
previewPlayType is a key in the bundle
ListPosition is a key in the bundle"

And from a podcast app named "podcast addict"

"
07-26 20:23:10.511: D/SLSBuiltInMusicAppReceiver(27211): Will read data from intent
07-26 20:23:10.511: D/DebugSLS(27211): duration is a key in the bundle
07-26 20:23:10.511: D/DebugSLS(27211): artist is a key in the bundle
07-26 20:23:10.512: D/DebugSLS(27211): playing is a key in the bundle
07-26 20:23:10.512: D/DebugSLS(27211): id is a key in the bundle
07-26 20:23:10.513: D/DebugSLS(27211): album is a key in the bundle
07-26 20:23:10.513: D/DebugSLS(27211): track is a key in the bundle
07-26 20:23:10.513: D/DebugSLS(27211): position is a key in the bundle
07-26 20:23:10.513: D/DebugSLS(27211): ListSize is a key in the bundle
"

@oleg-sh
Copy link

oleg-sh commented Aug 1, 2014

This is a bit of an issue for me too. SLS scrobbles podcasts from Podcast Addict app so I switched to AntennaPod and since recently it also began to scrobble audiobooks from Smart AudioBook Player...

@a-r-db
Copy link
Contributor

a-r-db commented Jun 30, 2016

I might be able to do this with Dikamilo's #50

@a-r-db
Copy link
Contributor

a-r-db commented Jul 16, 2016

After researching, I don't think there is a method for obtaining which pkg is using audio so no way to control scrobbling except for filters.

@a-r-db a-r-db added the bugs Functionality is broken. Ambiguous. label Aug 10, 2019
@a-r-db a-r-db added this to Core Feature Requests in Simple Last.fm Scrobbler Bug Triage Aug 11, 2019
@a-r-db
Copy link
Contributor

a-r-db commented Aug 29, 2019

#478 <- see here for the patched code
#479 <- see here for most recent released signed debuggable apk.

@a-r-db a-r-db closed this as completed Aug 29, 2019
Simple Last.fm Scrobbler Bug Triage automation moved this from Core Feature Requests to Closed Aug 29, 2019
@a-r-db a-r-db reopened this Aug 29, 2019
Simple Last.fm Scrobbler Bug Triage automation moved this from Closed to Needs triage Aug 29, 2019
@a-r-db a-r-db closed this as completed Aug 29, 2019
Simple Last.fm Scrobbler Bug Triage automation moved this from Needs triage to Closed Aug 29, 2019
@a-r-db
Copy link
Contributor

a-r-db commented Aug 29, 2019

There is no clear cut criteria for detecting podcasts.
I can however build a user generated words or symbols blacklist for tracks.
I will add it to my developer todo list.
PanoScrobbler Does Some Similar Handling

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugs Functionality is broken. Ambiguous. feature request optional issues
Development

No branches or pull requests

7 participants