erland;309207 Wrote: 
> Logitech regularly looks through bug/enhancement reports with provided
> patches and checks if the patch is good enough. If there is an
> enhancement, they also decide if they like to add it to SqueezeCenter
> and thus also manage the potential support issues later related to it.
> If they decide it's worth adding the decide which release they like to
> add it to. Usually critical bug corrections is added to the next fix
> release (7.0.x in this case), small enhancement is added to the next
> major release (7.1 in this case) and larger enhancements is added to a
> future release after the next major release (7.2 or later in this
> case). I suspect this is something that might be added to 7.1 or 7.2 if
> they decide they like to manage the support issues for this new option.
> 
> The only good way that end users can affect this decision is to vote on
> the registered enhancement request at http://bugs.slimdevices.com

right on.  i can't imagine the support burden of an "on/off" option is
too much.  and more to the point, SC is the only app i have ever seen
with such functionality as VA logic detection.

in other words, this kind of feature would be new to new users, and so
for new users, it should probably be off by default.  otherwise they
may be confused, as i was, as to what the hell was going on at first. 
if anything, i think having it off for new users by default will
lighten the support burden.

erland;309207 Wrote: 
> I have no idea, but I'm sure someone has already mention how it works in
> a post earlier in this thread (or in one of the related threads).

i'm guessing it does matter, but i can't remember why.  i am hoping
someone will explain why it matters if SC knows whether something is a
compilation or not.  i'm not saying it doesn't matter, i just couldn't
explain why it does if someone asked me.

erland;309207 Wrote: 
> If I'm reading the code correctly, I think the only albums that will be
> considered compilation with the new option disabled is those with a
> COMPILATION tag. I'm not sure what will happen with albums with an
> album artist named "Various Artists".

right...  it'll take some experimenting to see if certain strings in
tags or folder names cause one to be considered a comp by SC.  my guess
is that SC won't look at strings or folder names with the VA logic
turned off, but its just a guess.

erland;309207 Wrote: 
> If you like to help, you can verify how it works with and without the
> patch. 
> 
> If you are using Windows, you will need to install ActiveState perl and
> run slimserver.pl manually to be able to use the patch after you have
> applied it. If you feel this is to complicated, it would also help if
> you could just verify how this work today (without the patch).

unfortunately i don't have the skills (or recently, the time) to figure
out how to do perl and patch SC and so on...  is it possible for you to
send me the files that i could just copy over my existing files in my
SC install to make this work?  or am i just revealing how little i
know?

as to how it works now, i will happily answer any questions on that
score, but i don't know what you want me to say?  i'll take a stab at
it and if i am missing the point of your question just please ask for
clarifications:

i have mp3s, and they don't use user defined tags.  so, no album artist
tags, no comp tags, no tags other than the ones in the 2.3 spec.  (i
posted a grouping of four such files to bug 8001)

therefore the problem is that whenever any album i have has a TPE1
mismatch on even one track, it gets classified as a compilation (or so
i think) and it then sorts under Various Artists.  (artwork {gallery}
view, home -> albums -> artist, year, album)

in my case, about half of what the VA logic detects it detects
correctly, and the other half is incorrect.  it is my opinion that the
marketplace of mp3 users, (especially potential customers to slim) are
in the same boat.  and therefore they will be turned off greatly by
this unexpected behavior.

this all relates to bug 8001 as well, which i think is more important
than this issue.  would you be willing to provide a patch for that
issue as well?

essentially, the most popular proposed solution to that is to simply
provide another on/off option, and it would work thusly:

off: no changes to current scanner behavior

on: TPE2 will now populate the internal to SC7 field of ALBUMARTIST
instead of BAND.

http://bugs.slimdevices.com/show_bug.cgi?id=8001

this fix would defintely require testing to see if it works as
anticipated, but it isn't the only suggestion as to a possible fix, its
just the one seemingly "most popular" at this time b4 anything has been
actually tried yet.

thanks again for your efforts and please let me know what i can do, or
clarify.  thx!


-- 
MrSinatra

www.LION-Radio.org
Using:
Squeezebox2 (primary) / SBR (secondary) / SBC - w/SC 7.1beta - Win XP
Pro SP3 - 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=47297

_______________________________________________
discuss mailing list
discuss@lists.slimdevices.com
http://lists.slimdevices.com/lists/listinfo/discuss

Reply via email to