> 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
_______________________________________________ mythtv-dev mailing list [email protected] http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
