mopidy-local icon indicating copy to clipboard operation
mopidy-local copied to clipboard

Add "Album Artists" to root directories

Open tkem opened this issue 11 years ago • 9 comments

It's already supported in v0.8.0, just needs to be included in the default configuration:

directories =
    Albums              local:directory?type=album
    Album Artists       local:directory?type=artist&role=albumartist
    Artists             local:directory?type=artist
[...]

Some people may prefer it to the way "Artists" are handled, and it might help with support issues, e.g. "artist" tags vs. "albumartist" tags. Note that MPDroid also has a "Use album artists" setting for its local browse configuration.

tkem avatar Nov 26 '14 05:11 tkem

First, browsing "Album Artists" should give proper "local:album:" URIs...

tkem avatar Dec 01 '14 18:12 tkem

Also re-think get_distinct vs. browse URIs...

tkem avatar Mar 15 '15 07:03 tkem

@tkem I'm particularly interested in this issue as it'll make the Iris frontend a bit more versatile. Has any progress been made on this?

jaedb avatar Jun 09 '18 19:06 jaedb

@jaedb: As mentioned initially, this should already be possible (with some quirks, maybe) by manually adding it to the configuration. Otherwise, no progress on this, since I'm not actively using mopidy-local-sqlite myself any more and nobody else jumped on it. Maybe we should add a "maintainer wanted" sign up there, but that's why this got transferred to the "mopidy" org in the first place, IIRC.

tkem avatar Jun 09 '18 19:06 tkem

@tkem a belated follow-up question: if you're not using Mopidy-Local-SQLite, what backend extension are you using?

jaedb avatar Nov 11 '19 01:11 jaedb

@jaedb: I've been using mopidy-dleyna with the minidlna media server for the last couple of years.

tkem avatar Nov 12 '19 09:11 tkem

@tkem looking at the source code, it seems that this filter will only display tracks where albumartist is explicitly defined https://github.com/mopidy/mopidy-local/blob/5e6d26032c36220db646536a41066bf92c3d0b87/mopidy_local/schema.py#L69 therefor, since most single-artist album tracks are unlikely to have this definition, the usability of this filter is questionable.

would it be possible to rewrite this query so it doesn't exclude tracks, but rather reverts to artist field, if no albumartist is defined? I think this would be more useful; it would mean that compilation albums would be listed based on their albumartist field, and would ALSO list single-artist album tracks as well.

thank you

amitkeret avatar Sep 15 '20 07:09 amitkeret

@amitkeret: It's been some time since I've been using this, but IIRC most tracks (at least the ones I've encountered) will also provide an albumartist tag, which usually matches artist unless it's part of a compilation album.

tkem avatar Sep 15 '20 08:09 tkem

I can confirm that my artist list is significantly filtered if I use role=albumartist, meaning that the tracks do not provide/fallback to providing albumartist tag if this is not explicitly defined @jodal any ideas?

Mopidy 3.0.2 Mopidy-Local 3.1.1

amitkeret avatar Sep 16 '20 11:09 amitkeret