[mythtv-commits] mythtv commit: r11827 by rtsai

[mythtv at cvs.mythtv.org]

      Author: rtsai
        Date: 2006-11-27 06:23:30 +0000 (Mon, 27 Nov 2006)
New Revision: 11827
   Changeset: http://cvs.mythtv.org/trac/changeset/11827

Modified:

   trunk/mythtv/programs/mythcommflag/BlankFrameDetector.cpp
   trunk/mythtv/programs/mythcommflag/BorderDetector.cpp
   trunk/mythtv/programs/mythcommflag/BorderDetector.h
   trunk/mythtv/programs/mythcommflag/CannyEdgeDetector.cpp
   trunk/mythtv/programs/mythcommflag/CannyEdgeDetector.h
   trunk/mythtv/programs/mythcommflag/EdgeDetector.cpp
   trunk/mythtv/programs/mythcommflag/EdgeDetector.h
   trunk/mythtv/programs/mythcommflag/FrameAnalyzer.cpp
   trunk/mythtv/programs/mythcommflag/FrameAnalyzer.h
   trunk/mythtv/programs/mythcommflag/TemplateFinder.cpp
   trunk/mythtv/programs/mythcommflag/TemplateFinder.h
   trunk/mythtv/programs/mythcommflag/TemplateMatcher.cpp

Log:

- Fix bounding_box to be a little smarter; after doing the semi-binary-search
  for a candidate logo, re-expand to pick up all template pixels. Prior to this
  change, this routine always chopped off the edges of logos since they didn't
  contribute much to the "edginess" score. This bounding_box routine is still a
  hack (won't detect full-width templates, like with football games), but it's
  a better hack.

- Don't consider the middle 50% of the frame when searching for a logo. This
  helps avoid false positives in "talking-head" types of shows where the anchor
  wears a dark tie / white shirt / dark jacket, where these semi-static edges
  appear just as often and just as fixed as the "real" channel logo (suggested
  by dwalton at cisco dot com).

- Tweak some constants. Things now work better with some noiser analog
  recordings (dwalton at cisco dot com).

- Re-express all rectangles as (row,col,width,height) instead of
  (row,col,maxrow,maxcol) to avoid any kind of off-by-one confusion as to
  whether (maxrow,maxcol) is inclusive.

- Increase the amount of time that a logo may be delayed after a commercial
  break (from 25 to 60 seconds), or may disappear before a commercial break.

- Fix a corner-case bug in BlankFrameDetector that resulted in negative-length
  commercial breaks (and infinite loops).