[mythtv-commits] mythtv branch master updated by jpoet. v0.28-pre-3085-g125f6fe

Git Repo Owner noreply at mythtv.org
Sat Sep 26 23:15:46 UTC 2015


The branch, master has been updated on the
mythtv repository by gitolite user jpoet.
       via  125f6fe5796495634efb13dd14468da2fa9b8df1 (commit)
       via  90e378a8ed65c291053523956d198c3088f6f0df (commit)
       via  117af1f1f7a9195cb1a933f08c3c98082190945f (commit)
       via  7da9166e9d7cd62b68cd799e68411c8811e81446 (commit)
       via  c2d0e1cdd1411daa31b485466d0e51413256d176 (commit)
       via  990e96f89fbdc44666860d62a211a64dd9c7596a (commit)
      from  20603add41a1ee9eaa85cd11b8e73f3487eca765 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 125f6fe5796495634efb13dd14468da2fa9b8df1
Author:    faginbagin <mythtv at hbuus.com> at Sun, 26 Oct 2014 14:01:04 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:14:08 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=125f6fe5796495634efb13dd14468da2fa9b8df1

Mythcommflag assumed that video can be either normal, letter-boxed or pillar-boxed, but not both letter-boxed and pillar-boxed. I have found there are often instances where it can be both letter-boxed and pillar-boxed, especially in commercials found in SD recordings and on the SD version of our PBS channel. The patch, mcf-letter-pillar.patch changes the enum frameFormat into a bit mask allowing the detection of video that's both letter and pillar-boxed. This patch did not make any overall change to accuracy, at least not with the set of recordings I chose to measure results. But it didn't make things worse, and I do have other recordings where it helps. I just wasn't prepared to go back and repeat all the tests with a different set of recordings.
Signed-off-by: John Poet <jpoet at mythtv.org>



commit 90e378a8ed65c291053523956d198c3088f6f0df
Author:    faginbagin <mythtv at hbuus.com> at Sun, 26 Oct 2014 14:01:04 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:12:25 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=90e378a8ed65c291053523956d198c3088f6f0df

Logo detection required a minimum number of pixels in a mask found by examining several video frames to be greater than 50 to be considered a valid logo mask. No consideration of the variety of video resolutions was taken into consideration. And the 50 pixel minimum was not adjusted when lowres decoding was implemented. The patch, mcf-logo-pixInMask.patch, assumes the 50 pixel value was established for 1280x720 video and uses that area to compute a minimum pixel value using this formula: 50 * (width*height) / (1280*720 / 16). It improves accuracy to 75%.
Signed-off-by: John Poet <jpoet at mythtv.org>



commit 117af1f1f7a9195cb1a933f08c3c98082190945f
Author:    faginbagin <mythtv at hbuus.com> at Sun, 26 Oct 2014 14:01:04 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:05:18 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=117af1f1f7a9195cb1a933f08c3c98082190945f

The CommDetectBorder? setting did dual duty, as it was also used to limit the area examined by logo detection. The mcf-border.patch for problem 2) did not help a number of SD recordings, because it allowed logo detection to examine letter-boxed areas of SD video and caused logo detection to fail on recordings where it used to work. To detect logos, we need to exclude letter-boxed areas from SD video, but if we exclude too much from HD video, we'll miss the logo. Using the same border of 16 for both SD & HD with no scaling seems to be a good compromise. The patch, mcf-logo-border.patch, adds a new setting, CommDetectLogoBorder? with a default value of 16 and uses it as the border for logo detection. It improves accuracy to 72%.
Signed-off-by: John Poet <jpoet at mythtv.org>



commit 7da9166e9d7cd62b68cd799e68411c8811e81446
Author:    faginbagin <mythtv at hbuus.com> at Sun, 26 Oct 2014 14:01:04 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:03:34 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=7da9166e9d7cd62b68cd799e68411c8811e81446

Many years before the lowres option was added to improve performance, a default value for the CommDetectBorder? setting was set to 20. It is used to exclude CommDetectBorder? pixels from the top, bottom, left and right of the frame from examination. It should, for example, exclude any VBI data that might be visible at the top of a frame or parts of the video that would be hidden due to overscan. The default value was never adjusted to account for lowres decoding, nor did it take into consideration the variety of video widths and heights that exist today. The patch, mcf-border.patch, assumes the CommDetectBorder? setting was established for 720 height video and uses those two numbers and the lowres video height to compute the border used. For example: If video source height = 480 then border = 20 * 480 / 4 / 720 = 2 If video source height = 720 then border = 20 * 720 / 4 / 720 = 5 If video source height = 1080 then border = 20 * 1080 / 4 / 720 = 7 mcf-border.patch improves acc
 uracy to 67%.
Signed-off-by: John Poet <jpoet at mythtv.org>



commit c2d0e1cdd1411daa31b485466d0e51413256d176
Author:    faginbagin <mythtv at hbuus.com> at Sun, 26 Oct 2014 14:01:04 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:03:33 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=c2d0e1cdd1411daa31b485466d0e51413256d176

Mythcommflag assumed that the number of bytes per scan line for the luma array equals the video width. That may have been true before mythcommflag was changed to use ffmpeg's lowres decoding option, but it is not true when the lowres option is used. The bytes per scan line are actually the width rounded up to a multiple of 16. Some of the SD resolutions, when divided by 4 (as is the case for the lowres option), are not multiples of 16. The patch, mcf-bytesPerLine.patch, fixes this problem, and improved accuracy from 51% to 54%.
Signed-off-by: John Poet <jpoet at mythtv.org>



commit 990e96f89fbdc44666860d62a211a64dd9c7596a
Author:    John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:02:55 -0600
Committer: John Poet <jpoet at mythtv.org> at Sat, 26 Sep 2015 17:02:55 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=990e96f89fbdc44666860d62a211a64dd9c7596a

TVRec::TuningNewRecorder: If we fail to create the new recording, set status to Failed.



-----------------------------------------------------------------------

Summary of changes:
 mythtv/libs/libmythtv/tv_rec.cpp                   |    1 +
 .../programs/mythcommflag/ClassicCommDetector.cpp  |   72 +++++++++++++-------
 mythtv/programs/mythcommflag/ClassicCommDetector.h |    2 -
 .../programs/mythcommflag/ClassicLogoDetector.cpp  |   66 +++++++++++-------
 mythtv/programs/mythcommflag/ClassicLogoDetector.h |    2 +-
 .../mythcommflag/ClassicSceneChangeDetector.cpp    |    2 +-
 .../mythcommflag/ClassicSceneChangeDetector.h      |    2 +-
 mythtv/programs/mythcommflag/Histogram.cpp         |    9 ++-
 mythtv/programs/mythcommflag/Histogram.h           |    4 +-
 mythtv/programs/mythcommflag/LogoDetectorBase.h    |    3 +-
 .../mythcommflag/SceneChangeDetectorBase.h         |    4 +-
 11 files changed, 106 insertions(+), 61 deletions(-)

-- 



More information about the mythtv-commits mailing list