Opened 13 years ago

Closed 13 years ago

#2745 closed enhancement (fixed)

Subtitle parsing for RTL Group EIT

Reported by: l-case@… Owned by: Stuart Auchterlonie
Priority: minor Milestone: 0.21
Component: mythtv Version: 0.20
Severity: medium Keywords: DVB EIT RTL
Cc: Ticket locked: no

Description

This is a patch wich parse the description and finds the subtitle's if available. Patch was made with help from Janne

Attachments (2)

untertitel.diff (3.7 KB) - added by l-case@… 13 years ago.
2745_non_fuzzy.diff (5.7 KB) - added by Janne <janne-mythtv@…> 13 years ago.

Download all attachments as: .zip

Change History (9)

Changed 13 years ago by l-case@…

Attachment: untertitel.diff added

comment:1 Changed 13 years ago by Stuart Auchterlonie

Milestone: unknown0.21
Owner: changed from Isaac Richards to Stuart Auchterlonie

comment:2 Changed 13 years ago by Stuart Auchterlonie

Resolution: fixed
Status: newclosed

(In [12139]) Closes #2745. Adds EIT fixup for RTL Group DVB by applying slightly modified version of the patch.

Thanks to l-case at gmx dot net for the original patch.

comment:3 Changed 13 years ago by Stuart Auchterlonie

(In [12141]) Refs #2745. The conversion of fixup from unsigned char to uint32_t is important.

Changed 13 years ago by Janne <janne-mythtv@…>

Attachment: 2745_non_fuzzy.diff added

comment:4 Changed 13 years ago by Janne <janne-mythtv@…>

Resolution: fixed
Status: closedreopened
Type: patchenhancement

the commited fixup has way too much false positives. The 2745_non_fuzzy.diff implements subtitle extraction based on regular expressions for episode numbers or quoted subtitles. It works well for VOX and Super RTL but not only on a few RTL and RTL2 programs.
I think unprocessed descriptions are less disturbing than false positive. I can't see a good way to avoid false positives for RTL and RTL2. The best approach I can think of atm is program length > 20 and < 65 minutes and number of words in the first sentence at most 3 or 4.

comment:5 Changed 13 years ago by Janne <janne-mythtv@…>

2745_non_fuzzy.diff is based on another patch from l-case at gmx dot net

comment:6 Changed 13 years ago by Stuart Auchterlonie

(In [12247]) Refs #2745. Updates to the RTL fixups from Janne and l-case at gmx dot net

comment:7 Changed 13 years ago by Janne Grunau

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.