Opened 8 years ago

Closed 5 months ago

Last modified 5 months ago

#10938 closed Patch - Bug Fix (fixed)

Logo detection not working

Reported by: lomion@… Owned by: Mark Kendall
Priority: minor Milestone: 31.0
Component: MythTV - Mythcommflag Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I set the commercial detection to only detect Logos. This works best for german TV Station as long as they a broadcasting HDTV (DVB-S2/H.264). For PAL-SD recordings (DVB-S/MPEG-2) mythcommflag returns successfully but always with 0 commercial breaks.

I would be happy to provide traces if you tell me how this is done in mythcommflag ...

Attachments (1)

10938.patch (735 bytes) - added by lomion@… 8 years ago.

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by lomion@…

Attachment: 10938.patch added

comment:1 Changed 8 years ago by lomion@…

Problem is related with the low-res feature of ffmpeg ... the attached patch solves the problem (detection results are still not perfect but at least I get some ;-) )

I can provide traces if needed ...

comment:2 Changed 8 years ago by Kenni Lund [kenni a kelu dot dk]

Owner: changed from cpinkham to beirdo
Status: newassigned
Type: Bug Report - GeneralPatch - Bug Fix

comment:3 Changed 6 months ago by Mark Kendall

Milestone: unknown31.1
Owner: changed from beirdo to Mark Kendall
Status: assignedaccepted

comment:4 Changed 6 months ago by Mark Kendall

Version: 0.25-fixesMaster Head

comment:5 Changed 5 months ago by Mark Kendall <mark.kendall@…>

Resolution: fixed
Status: acceptedclosed

In a4c884910e/mythtv:

Actually use 'Enable experimental speedup of advert detection' setting

  • at some point in the dim and distant past this was broken - and the

'experimental' aspects are used by default (i.e. the setting value is
ignored)

Fixes #10938

comment:6 Changed 5 months ago by Mark Kendall <mark.kendall@…>

In 149f389c64/mythtv:

Actually use 'Enable experimental speedup of advert detection' setting

  • at some point in the dim and distant past this was broken - and the

'experimental' aspects are used by default (i.e. the setting value is
ignored)

Fixes #10938

(cherry picked from commit a4c884910e681c563edc9c0755f2a12f75888abf)

comment:7 Changed 5 months ago by Stuart Auchterlonie

Milestone: 31.131.0
Note: See TracTickets for help on using tickets.