MrSinatra;290068 Wrote: > here is what i would suggest: > > the word "list" be removed. display is not a good word either, as that > also could imply sort. > > what i think is least problematic is: > > 1. Denote all artists on an album > and > 2. Denote albums as by band > > this methodology is flexible, meaning a second option can then be > created for sorting as a separate function. > > btw, the infobox currently says this: > > "Albums that contain songs that are tagged with a band may be listed > under that band name or with the other artists for that album. The band > tag is also known as TPE2 and may appear as the "album artist" in some > software." > > i would change it to: > > "You can choose what artist or tag information to use to show what > artist an album is by. The first choice uses TPE1 tags, the second > choice uses TPE2 tags. (TPE2 is also known as "Band" or "Album Artist" > in some other software)."
snarlydwarf;290074 Wrote: > The first choice is ALBUMARTIST. > > If this does not exist and the flag is set, use BAND > > Else if not COMPILATION, return list of artists. > > etc. MrSinatra;290075 Wrote: > i don't understand your point, you don't have much context to explain > what you are saying. and those are INTERNAL SC7 tags and users would > not understand the implication of that. can you please explain just > what you mean? snarlydwarf;290081 Wrote: > That is what the code does. That is the context. > > If you wish to change the help documentation, it should match what the > code does. i assume you mean the infobox, right? snarlydwarf;290081 Wrote: > . getting-artist-for-album-display > . if ALBUMARTIST return ALBUMARTIST > . if bandtag and BAND return BAND > . if !COMPILATION and !VA return list of ARTISTS (on all tracks) > > etc. so thats what the code looks like and thats what it does in actuality is what you are saying, right? keep in mind, i do not know how to write code, so the ! means nothing to me. and do you mean to say that the above is how it works when 'group comps together' and 'list albums by band' are both chosen? snarlydwarf;290081 Wrote: > You said that TPE1 (track artist) was checked first. It is not. > > ALBUMARTIST is. > > Then BAND. > > Then a variety of things dealing with VA stuff. > > But the code checks ALBUMARTIST first. Then BAND. when scanning you mean? or what? i did not btw say it was CHECKED first, i said the first CHOICE refers to the TPE1 tag. the second CHOICE refers to the TPE2 tag. please see above as i requoted it in this post. i think perhaps on rereading what i wrote, you won't object to my suggested changes on this option. snarlydwarf;290081 Wrote: > It is not TPE2. This is only executed long after the file has been > scanned. It affects all formats, not just MP3: ie, FLAC has no > commonly used tag called 'TPE2' but this will affect FLACs that have a > field named 'BAND'. i understand all that. but i don't see what i wrote as being confusing on that point. we agree that the option as is is for denoting the internal BAND tag. my infobox explanation will get that point across to users without steeping them in a technical discussion. snarlydwarf;290081 Wrote: > Again, this section of code does not have access to the names of the > tags (TPE1 or TPE2, etc), so it must be discussed in terms of the > internal names of the tags. Saying it would return TPE2 is blatantly > wrong: it would return no such thing on WMA, FLAC, APE, OGG, WAv, AIFF, > etc. It returns the BAND field, regardless of what the original tag > name was. the infobox as is already does not get into such issues as "internal SC7 tags." i'm pretty sure slim doesn't want it to. the scanner does in fact populate BAND with TPE2 tags, so my explanation is reasonable for a non-technical explanation. (again, the current explanation ALREADY mentions TPE2) and since it wouldn't change how it currently works, it wouldn't confuse people like you who already have it the way they want. it would be a transparent change. snarlydwarf;290081 Wrote: > Saying that it uses TPE1, therefore is wrong on two counts: the code is > cross-format and FLAC does not have a TPE1 yet is still subject to the > behavior of this code. the infobox already mentions TPE2, i see no problem with mentioning TPE1 as well, but if you like we could be more precise (in bold): "You can choose what artist or tag information to use to show what artist an album is by. The first choice uses TPE1 *(or Artist)* tags, the second choice uses TPE2 tags. (TPE2 is also known as "Band" or "Album Artist" in some other software)." snarlydwarf;290081 Wrote: > It is also wrong because TPE1/ARTIST is not checked first. indeed, good thing i didn't say that. -- MrSinatra www.LION-Radio.org Using: Squeezebox2 (primary) / SBR (secondary) / Jive - w/SC 7.0.1beta - Win XP Pro SP2 - 3.2ghz / 2gig ram - D-Link DIR-655 ------------------------------------------------------------------------ MrSinatra's Profile: http://forums.slimdevices.com/member.php?userid=2336 View this thread: http://forums.slimdevices.com/showthread.php?t=46093 _______________________________________________ discuss mailing list [email protected] http://lists.slimdevices.com/lists/listinfo/discuss
