[mythtv] New duplicate check method 'smart'

[martin at longhome.co.uk]

Ok. I've removed the word '(smart)' from the description, and changed it to
'Subtitle else Description'. I can go for 'or' if the general consensus
prefers, but unless it get's committed I guess it doesn't really matter. 

Anyone tried this yet? I've set it as my default method, and it seems to
work really well now for all my schedules (some were set to 'Subtitle' and
some to 'Description'). And it also works really well for those few cases
where the eit data is inconsistent. 

I'll also look at updating the wml and compact themes for mythweb. Is there
anything else specific that I need to look at? Anyone who know's the
scheduler code well think I've missed anything?

Original Message:
-----------------
Date: Sat, 4 Nov 2006 00:33:58 -0000
From: "Martin Long" <martin at longhome.co.uk>
Subject: Re: [mythtv] New duplicate check method 'smart'
To: <mythtv-dev at mythtv.org>
Message-ID: <001301c6ffa8$eafdd490$0501a8c0 at excelsior2>
Content-Type: text/plain;	charset="us-ascii"


Yeah, to be honest, I kind of though 'smart' was a bit pretentious and
ambiguous, although I have some trouble with 'or', because it's not strictly
speaking OR logic. Anyway, feel free to alter the strings to fit in with
what ever name you want to come up with. I kind of like something like
'Subtitle then Description' or 'Subtitle else Description'. 




-----Original Message-----

"smart" is a little ambiguous, and leaves room for change for someone
else's definition of "smart".

Why not just call it "Subtitle *or* description", to complement the
current "Subtitle *and* description"?


-----Original Message-----
From: Martin Long [mailto:martin at longhome.co.uk] 
Sent: 03 November 2006 22:45
To: 'mythtv-dev at mythtv.org'
Subject: New duplicate check method 'smart'

I've created a new 'smart' duplicate check method. I normally use either
subtitle, or description. Sometimes a series flips between using a subtitle,
and not (particularly with the UK eit guide data!). 'Subtitle and
description' is pretty useless for me, because a) if the subtitles are none
existent then programs don't match b) if subtitles are present they can be
fouled by small differences in the description - the whole reason for using
subtitle in the first place.

So, I created 'Subtitle then description (smart)'. This first checks the
subtitles match, however, if no subtitle exists, then and only then will it
check the description.

I've done a little testing, and it seems to work ok, and helps to iron out
some of the inconsistencies caused by the uk eit data. Anyway... see how it
goes. I've attached patches for trunk, but they apply to 20-fixes ok.

Mythweb patch also included, but you'll need to update your .lang file with
a string for 'Subtitle then Description (smart)'. I updated the english one,
but spooked the diff, so i left that out.


http://cvs.mythtv.org/trac/ticket/2655


Martin



--------------------------------------------------------------------
mail2web - Check your email from the web at
http://mail2web.com/ .