Bug 10760 - Various Artists listed as Album Artist / TPE2 setting ignored
: Various Artists listed as Album Artist / TPE2 setting ignored
Status: CLOSED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Controller
: 7.3.3
: PC Windows XP
: P3 normal with 4 votes (vote)
: 7.6.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-01-19 17:18 UTC by Ivan
Modified: 2011-05-23 16:43 UTC (History)
14 users (show)

See Also:
Category: Bug


Attachments
Screenshot of web interface, Squeezesoft, and Squeezeplay (76.07 KB, image/gif)
2009-01-20 12:19 UTC, Ivan
Details
FLAC file with the appropriate tags (21.28 MB, audio/x-flac)
2009-01-20 12:21 UTC, Ivan
Details
correct artist information on SP (57.79 KB, image/png)
2009-02-04 03:29 UTC, Michael Herger
Details
Screenshot of settings (19.77 KB, image/gif)
2009-02-05 13:58 UTC, Ivan
Details
Still a problem with latest Squeezeplay and 7.4.2 (86.52 KB, image/gif)
2010-01-23 12:47 UTC, Ivan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan 2009-01-19 17:18:56 UTC
Version: 7.3.2 - 24695 @ Sat Jan 17 03:01:48 PST 2009
Platform Architecture: 586
Perl Version: 5.8.8 - MSWin32-x86-multi-thread
MySQL Version: 5.0.22-community-nt

Squeezeplay is ignoring the "Treat TPE2 MP3 tag as Album Artist / List albums by band" setting in SqueezeCenter, and is listing multiple artist albums as Various Artists.

For example, I have the At Last album by Cyndi Lauper, and two of the tracks have guest artists (separated by ";"). So I enter a TPE2 for Album Artist as "Cyndi Lauper". On the my Squeezebox 3, web browser interface and SoftSqueeze, the album is listed as "At Last by Cyndi Lauper." But on SqueezePlay, it is listed as, "At Last by Various Artists."
Comment 1 James Richardson 2009-01-20 11:43:20 UTC
Ivan: does the same thing happen on the SBController (if you have one)

Can you provide a sample file so we can test here.
Comment 2 Ivan 2009-01-20 12:19:38 UTC
Created attachment 4674 [details]
Screenshot of web interface, Squeezesoft, and Squeezeplay

Here is the screen shot of how the album is shown in the web interface, Softsqueeze and Squeezeplay to illustrate.
Comment 3 Ivan 2009-01-20 12:21:42 UTC
Created attachment 4675 [details]
FLAC file with the appropriate tags

This is the FLAC track with the tags with multiple artists from the album.
Comment 4 Ivan 2009-01-20 12:23:34 UTC
Unfortunately, I don't have an SBController.
Comment 5 James Richardson 2009-01-20 14:04:48 UTC
What version of SqueezePlay are you using?  I tested with 7.3 r3729 ( but did not see this issue.

Please try the latest which is 7.3r3856

Also, you may want to try a Clear and Rescan on your library. 
Comment 6 Ivan 2009-01-20 14:13:06 UTC
Well, the windows distro does not show a version number, so there's no way for me to tell. But I just downloaded the one dated Jan 20, 2009 from http://downloads.slimdevices.com/nightly/latest/7.3/ and tried it. Same thing--shows the album as Various Artists.

My library is cleared and rescanned every night. A library anomaly doesn't explain why it displays properly on the web interface, the SB3 and SqueezeSoft. But NOT in SqueezePlay.
Comment 7 James Richardson 2009-01-20 14:44:11 UTC
Ben/Tom: Can you think of anything that would cause this in SP?
Comment 8 Ben Klaas 2009-01-20 14:48:11 UTC
Yes. The query to produce and deliver menus to Squeezeplay-driven interfaces is different from the Player UI queries.

This is a bad bug for me to fix-- I have next to no experience in tweaking SC databse queries.
Comment 9 Chris Owens 2009-01-26 09:20:34 UTC
Assume the (sb2/3) player UI is right
Comment 10 Michael Herger 2009-02-04 03:29:17 UTC
QA to reproduce. This works for me.

Ivan - what exact settings do you have for all the various compilation and artist options?
Comment 11 Michael Herger 2009-02-04 03:29:54 UTC
Created attachment 4759 [details]
correct artist information on SP
Comment 12 James Richardson 2009-02-05 09:29:33 UTC
*** Bug 10739 has been marked as a duplicate of this bug. ***
Comment 13 Joerg Schwieder 2009-02-05 10:17:55 UTC
As of 
https://bugs-archive.lyrion.org/show_bug.cgi?id=10739
Which was just marked as a duplicate of this I would like to add that I don't think this is a SqueezePlay (client) issue but a SC issue - the same thing is seen in iPeng so obviously the data delivered sometimes is wrong.
Comment 14 Ivan 2009-02-05 13:57:27 UTC
(In reply to comment #11)
> Created an attachment (id=4759) [details]
> correct artist information on SP
> 

Could this be because this is only one track? This occurs when there are tracks in a single album that has more than one artist listed with a separator and a TPE2 set to a single artist.
Comment 15 Ivan 2009-02-05 13:58:43 UTC
Created attachment 4770 [details]
Screenshot of settings

Here is a screenshot of the music library settings in slimserver.
Comment 16 Ivan 2009-02-05 14:00:42 UTC
(In reply to comment #13)
> As of 
> https://bugs-archive.lyrion.org/show_bug.cgi?id=10739
> Which was just marked as a duplicate of this I would like to add that I don't
> think this is a SqueezePlay (client) issue but a SC issue - the same thing is
> seen in iPeng so obviously the data delivered sometimes is wrong.
> 

Also happening in Moose 0.70
Comment 17 James Richardson 2009-02-05 15:45:01 UTC
*** Bug 10679 has been marked as a duplicate of this bug. ***
Comment 18 James Richardson 2009-02-11 06:45:55 UTC
*** Bug 11036 has been marked as a duplicate of this bug. ***
Comment 19 Chris Owens 2009-03-16 09:48:48 UTC
We are now planning to make a 7.3.3 release.  Please review your bugs (all marked open against 7.3.3) to see if they can be fixed in the next few weeks, or if they should be retargeted for 7.4 or future.

Thanks!
Comment 20 Chris Owens 2009-03-30 17:34:24 UTC
Since there's now a planned 7.3.3 release, bugs which won't make the cut-off are being moved to the next target out.  If you feel that this bug needs to be addressed more (or less) urgently than the 7.4 release, please cc chris@slimdevices.com and leave a comment in the bug to that effect so we can review it.

Thanks.
Comment 21 Don 2009-06-28 11:17:44 UTC
I am having the same issue but with the controller. I use SoftSqueeze and not SqueezePlay so I can't comment on the problem with that.

I have posted it on the forum at http://forums.slimdevices.com/showthread.php?p=436203#post436203 (user id Qdon). This issue was posted by firedog back in February but recevied no responses. Can this issue please get addressed.
Comment 22 Don 2009-06-28 11:18:47 UTC
Sory forget to mention that I am running the production version of 7.3.3.
Comment 23 James Richardson 2009-06-28 11:36:24 UTC
Changng product
Comment 24 Blackketter Dean 2009-07-22 08:40:00 UTC
Moving to the product SqueezePlay because this bug appears to apply to any player based on that application code.  Feel free to move it back if it's specific to the single original product.
Comment 25 Ben Klaas 2009-08-26 07:52:51 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 26 Mike Walsh 2009-08-27 23:51:49 UTC
ivan, do you set TPE2 ONLY for those two tracks, or for all the tracks on the CD?  (i have it too btw)

do you use the same separator character on other CDs?  how are their TPE2's set?

finally, have you done a full clear and rescan?
Comment 27 SVN Bot 2009-09-03 06:01:41 UTC
 == Auto-comment from SVN commit #28422 to the slim repo by michael ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28422 ==

Bug: 10760
Description: add 'timeout' preference to MusicIP, to be used in synchronous requests like the mix command.

Please note: as mentioned before this can lead to hiccups if you set this value too high and other players starve waiting for MIP to return. This preference isn't exposed in any UI, as using it might negatively influence SC behaviour. It's for you poweruser's (and my ;-)) convenience only. 

To set the parameter either:
- shut down SC
- add "timeout: 10" (or whatever seconds you want) to prefs/plugin/musicip.prefs
- restart SC

or :
- "telnet yourmachine 9090"
- type "pref plugin.musicip:timeout 10" (or whatever), hit enter

Up to you to find a value which fits you well.

BTW: I'm running MIP against a collection of about 15k tracks.
- main Linux server (1GHz Via C3) took more than a week(!) to analyze this. Mixing never was an issue with the default 5 seconds.
- test Windows Home Server (1.6GHz Core2 solo - what a nice name for a good ol'Celeron!) finished the analysis in a few hours, but times out on many mixes...
Comment 28 Michael Herger 2009-09-03 06:04:16 UTC
Oops... sorry, wrong bug.
Comment 29 Ivan 2009-09-04 11:05:31 UTC
(In reply to comment #26)
> ivan, do you set TPE2 ONLY for those two tracks, or for all the tracks on the
> CD?  (i have it too btw)
> 
> do you use the same separator character on other CDs?  how are their TPE2's
> set?
> 
> finally, have you done a full clear and rescan?

TPE2 is set for ALL the tracks in the CD.

Separator? You mean a ";" (semi-colon) for multiple genres or artists on a track? Yes--I consistently use ";". 

TPE2 is set only on CD's where there may be more than one artist on a given track/s. For instance, a duet or a featured guest artist. I set that track to include both the album artist and the guest artist/s separated by semi-colons. Then set the TPE2 for ALL tracks on that CD/album for the primary artist. 

Yes--I always do a full clear and rescan since there is another bug that won't display the album art correctly without a full rescan when changing/adding to my library.
Comment 30 Mike Walsh 2009-09-22 17:55:53 UTC
so is the thought here that the separator character is the cause of squeezeplay getting it wrong?

what happens on VA albums with TPE2 set where no tracks have used a separator character?
Comment 31 Ross Levine 2009-10-06 17:28:11 UTC
*** Bug 14634 has been marked as a duplicate of this bug. ***
Comment 32 tswrench 2009-10-14 14:10:28 UTC
This bug deals with the way 7.4 looks at tags and extended tag fields. Something has changed. The previous version did it correctly. Most of the music in my collection migrated over just fine... Why? Because the "Album Artist" field in the tag was not used or echoed the same name in the "Artist" field. For example, if "Beck" is the name in the Artist field, AND "Beck" is the name in the "Album Artist" field (or the Album Artist field is left blank), then the music is sorted correctly as "Beck" in Squeezebox. However, if the "Artist" field contains "Beck," and the "Album Artist" field contains "LaRon," then Squeezebox is going to sort it as "LaRon," and thus far I haven't found a way to make it filter differently. 

TEST CASE: 

- Rip a cd and enter the correct name in the "Artist" field 

- Enter an erroneous name in the "Album Artist" field 

The erroneous name is the one that will persist in Squeezebox.  Fix this bug so that the Artist listing keys on the "Artist" field AND NOT the "Album Artist" field.
Comment 33 Mike Walsh 2009-10-14 14:26:20 UTC
tswrench,

i'm not sure why you are posting that info to this bug, its not germane to the bug as far as i can see, but i can tell you a few things:

1. SBS insists on making "one master list" out of your files.
2. if AA tags are present, they will ALWAYS take precedence for sorting and denoting (over artist tags) in SBS in the master list.  don't expect that to change.
3. the master list terms of things like "artist" are not a 1 for 1 match with the file tag called "artist."  again thats by design.
4. i actually agree w/SBS that AA tags should take precendence over simple artist tags if you have to pick one for a single master list, HOWEVER i really hate the way SBS insists on one master list and doesn't make it easy to switch between browsing via AA or artist, or whatever, it makes you do it ITS way, period, and changes frequently involve rescans or simply can't be done at all.  not a good thing imo.

i think you should move this discussion of yours to a forum thread.  jmho.
Comment 34 Ivan 2010-01-23 12:47:50 UTC
Created attachment 6456 [details]
Still a problem with latest Squeezeplay and 7.4.2

I just tested this with the latest SqueezePlay version 7.4 r0 and Squeezebox Server 7.4.2 r29707. Problem is still there.

I hope this bug isn't on the Squeezebox Touch when it is released! Or get it fixed before the launch! Because the current version of SqueezePlay looks like a "Touch emulator."
Comment 35 Mike Walsh 2010-01-23 13:37:22 UTC
(In reply to comment #30)
> so is the thought here that the separator character is the cause of squeezeplay
> getting it wrong?
> what happens on VA albums with TPE2 set where no tracks have used a separator
> character?

in addition to those questions, you should probably test the 7.5embedded branch as thats whats currently being worked on.  get it in the ../nightly/ root.
Comment 36 Ivan 2010-01-23 14:09:08 UTC
(In reply to comment #35)
> (In reply to comment #30)
> > so is the thought here that the separator character is the cause of squeezeplay
> > getting it wrong?
> > what happens on VA albums with TPE2 set where no tracks have used a separator
> > character?
> 
> in addition to those questions, you should probably test the 7.5embedded branch
> as thats whats currently being worked on.  get it in the ../nightly/ root.


I wasn't insinuating that the separator was the cause--I don't know what the cause is. You had asked whether I used a separator, and my reply is, "Yes--I use a semi-colon!" There is no other way to list multiple artists on a track, but have them treated as individual artists, other than to use a separator. If I use "Bob and Mary" as the artist name instead of a separator, then a new artist "entity" is created, and the track won't be listed individually for Bob, and then for Mary.

Which answers the second question--I wouldn't want to create a "Bob and Mary" artist by omitting the separator for the Track Artist. The end goal is that there are many albums that are Bob's albums, but he may have guest artists on one or a few tracks on that album. That album is NOT a "Compilation" album, and shouldn't be treated as such.

In essence, I thought that's what the Album Artist/TPE2 setting is for. It's to allow for non-compilation albums that have multiple artist performing on the individual tracks. 

If it's a Compilation album or Various Artist album, the the Album Artist/TPE2 setting would be left blank. Whether it should be blank on an album where all tracks contain only one artist name is arguable. but the default logical behavior when Album Artist is blank should be:

1. All the tracks in the album have the same artist. Therefore, the Album Artist equals the track artist.

2. Tracks have multiple artists, therefore this is a "Compilation/Various Artists" album.

The problem is that only Squeezeplay and Moose handle this incorrectly--both the web interface and Softsqueeze handle it correctly.
Comment 37 Chris Owens 2010-03-09 17:58:41 UTC
The 7.4.x target is in the past
Comment 38 Mike Walsh 2010-11-10 23:24:05 UTC
Ivan,

is this still a problem for you?

i don't see it on the 7.5.2 beta, but i might be reproducing it wrong.

can you give explicit step by step instructions to reproduce?
Comment 39 Andy Grundman 2011-01-12 11:58:16 UTC
This should probably be looked at in onebrowser.
Comment 40 Philip Meyer 2011-01-12 14:25:35 UTC
(In reply to comment #3)
> Created an attachment (id=4675) [details]
> FLAC file with the appropriate tags
> 
> This is the FLAC track with the tags with multiple artists from the album.

I've just read through this bug report again, and perhaps a crucial thing that hasn't been mentioned yet that looks very likely to be a cause is the fact that the bug report discusses "TPE2", but the example problem file is a FLAC file?

So I downloaded the example flac file attachment, and opened it in Mp3Tag.  The file only has FLAC tags - no id3 tags, and thus there isn't a TPE2(BAND) tag to be read.

In the FLAC tags, there is a BAND tag, and no "ALBUM ARTIST" or "ALBUMARTIST" tag, so this is clearly why it doesn't work.

SBS is not ignoring the TPE2 setting or tag because there isn't one!

The fix: open your album in Mp3Tag, go into Extended Tags, and double-click the tag called BAND and change the name of the tag to "ALBUM ARTIST".
Comment 41 Mike Walsh 2011-03-06 10:39:11 UTC
nice catch Phil, and i agree, this bug should be closed, unless Ivan or someone objects for some reason i don't see.
Comment 42 Andy Grundman 2011-04-07 10:20:46 UTC
Closing as suggested.