[mythtv] [PATCH] Fix for commercial flagging using logo detection

Chris Pinkham cpinkham at bc2va.org
Fri Feb 18 06:06:59 UTC 2005


> Looks like some code is missing from commercial_skip.cpp, so it is not
> possible to flag commercials using logo detection only. The attached
> patch should fix this.

Thanks.

Not sure how I missed getting that back in when I rearranged things when
I created the ALL method.  It looks like it implements it the way I
had the code before so this has been committed to CVS.

> The patch also adds some values to the edgeDiffs[] array. I have
> experienced needing a value as high as 50 to detect the logo area.

This is in there also, I don't beleive this can cause false positives.
Are these logos transparent or opaque?  I never even tried bumping it
up that high, maybe that's why I wasn't getting results as good as I
hoped when testing opaque logos.


-- 

Chris



More information about the mythtv-dev mailing list