Opened 4 years ago
Closed 3 years ago
Last modified 3 years ago
#13016 closed Patch - Bug Fix (fixed)
Missing Teletext Subtitles in Recordings
Reported by: | Owned by: | Peter Bennett | |
---|---|---|---|
Priority: | minor | Milestone: | 29.1 |
Component: | MythTV - Captions | Version: | 0.28.1 |
Severity: | medium | Keywords: | teletext subtitles captions |
Cc: | Peter Bennett | Ticket locked: | no |
Description
I notice a lot of missing CC text from Teletext Subtitles in DVB-C recordings independent of the recorded channel.
VLC, ccextractor, FFmpeg 3.2.4 show all CC text correct. MythTV player and mythccextractor miss almost every second/third CC text. In a recording Myth found only 732 of 1651 total.
Inspecting the ccextractor generated srt file doesn’t show any timing problems, which could be misinterpreted by MythTV.
I tried VBI debugging, but no CC text was logged.
Also no “subtl timecode” from libs/libmythtv/avformatdecoder.cpp AvFormatDecoder::ProcessSubtitlePacket? although proper debugging options were set.
Appreciate any pointers where to add some debugging to help.
Attachments (2)
Change History (18)
comment:1 Changed 4 years ago by
comment:2 Changed 4 years ago by
Cc: | Peter Bennett added |
---|
Changed 3 years ago by
Attachment: | 0001-Teletext-Magazin-Serial-Mode-implemented.patch added |
---|
comment:3 Changed 3 years ago by
The above patch fixes the missing teletext subtitle problem, it implements "Teletext Magazin Serial Mode".
Only Parallel Mode was implemented, therefor some subtitles where not displayed when there where not enough teletext page changes in the magazin where the subtitle pages where gathered. In Magazin Serial Mode any new page finishes the previously gathered teletext pages independently of the magazin of the new page.
comment:4 Changed 3 years ago by
Hi Angela, on all channels I have perfect subtitles now. Great job, thanks for the fix. Karl
comment:5 Changed 3 years ago by
Hi, I am assuming that as this bug report has not been closed, you are still working on it. Also that the code also applies to CEA 608/708 subtitles / closed captions on DVB-T recordings. If either is not the case, I can submit another bug specific to DVB-T.
Our family has noted, in DVB-T recordings, all the same behavior discussed in the original thread, although we have not recently noticed the problem fixed in your patch.
In addition, some recordings captions are garbled/strange/poorly decoded(?). I have posted a ~1 minute/ ~150Mb sample file http://www.gigasize.com/get/two5x6jrn7b that shows myth seems to have difficulty with 708 captions on recordings. The two srt files (mythccextractor) http://www.gigasize.com/get/mbft418ywmc 608 srtfile http://www.gigasize.com/get/n005g7b2c0c 708 srt file.
This problem seems to cascade/include other behaviour, which might be related, but I do not yet have a good sample for any of those.
Wayne
Current mythtv version info - although this was recorded with a slightly earlier version of myth.
MythTV Version : v0.28.1-38-geef6a48 MythTV Branch : fixes/0.28 Network Protocol : 88 Library API : 0.28.20161120-1 QT Version : 5.5.1 Options compiled in: linux profile use_hidesyms using_alsa using_oss using_pulse using_pulseoutput using_backend using_bindings_perl using_bindings_python using_bindings_php using_crystalhd using_dvb using_firewire using_frontend using_hdhomerun using_vbox using_ceton using_hdpvr using_ivtv using_joystick_menu using_libcec using_libcrypto using_libdns_sd using_libfftw3 using_libxml2 using_lirc using_mheg using_opengl using_opengl_video using_opengl_themepainter using_qtwebkit using_qtscript using_qtdbus using_sdl using_taglib using_v4l2 using_x11 using_xra ndr using_xv using_profiletype using_bindings_perl using_bindings_python using_bindings_php using_freetype2 using_mythtranscode using_opengl using_vaapi using_vdpau using_ffmpeg_threads using_mheg using_libass using_libxml2
Vlc displays the CEA 608 captions in the original 18+ Gb file, but apparently didn’t recognize the presence of the second channel. The posted portion was cut with avidemux, using the mpg ts demuxer as that seems to be keeping the extra streams intact.
comment:6 Changed 3 years ago by
@weworama
The patch fixes the original problem about missing subtitles. It was fixed by implementing "Teletext Magazin Serial Mode". The original issue is solved and I hope to see it integrated in the next Mythtv release.
Please create a new issue for your garbled captions.
comment:7 Changed 3 years ago by
Owner: | changed from Jim Stichnoth to Peter Bennett |
---|---|
Status: | new → assigned |
I would like to validate the patch. There are no teletext subtitles where I live, so I downloaded some files with teletext magazine subtitles. I do not see any missing subtitle when I play it on current master. Can you supply a test file or tell me what to do to see the problem.
TS Reader Lite shows this on two files I downloaded and the subtitles seem to be fine on existing MythTV master:
Elementary Stream PID 258 (0x0102) Teletext/VBI Descriptor: Teletext Descriptor Language: und Type: Initial Teletext page Magazine: 0x01 Page: 0x00 Language: swe Type: Teletext subtitle page Magazine: 0x01 Page: 0x99 Language: nor Type: Teletext subtitle page Magazine: 0x02 Page: 0x99 Language: dan Type: Teletext subtitle page Magazine: 0x05 Page: 0x99 Language: fin Type: Teletext subtitle page Magazine: 0x06 Page: 0x99 Language: swe Type: Initial Teletext page Magazine: 0x01 Page: 0x01 Language: nor Type: Initial Teletext page Magazine: 0x02 Page: 0x01 Language: dan Type: Initial Teletext page Magazine: 0x05 Page: 0x01 Language: fin Type: Initial Teletext page Magazine: 0x06 Page: 0x01
Elementary Stream PID 5303 (0x14b7) Teletext/VBI Descriptor: Stream Identifier Descriptor 04 . Descriptor: Teletext Descriptor Language: eng Type: Teletext subtitle page Magazine: 0x00 Page: 0x88 Language: eng Type: Initial Teletext page Magazine: 0x00 Page: 0x88 Descriptor: Private Data Indicator Descriptor 4f 54 56 00 OTV. Descriptor: User Private Descriptor: 0xfe 53 55 42 00 SUB.
comment:8 Changed 3 years ago by
Status: | assigned → infoneeded |
---|
comment:9 Changed 3 years ago by
Type: | Bug Report - General → Patch - Bug Fix |
---|
Changed 3 years ago by
Attachment: | 0001-Teletext-Magazin-Serial-Mode-implemented-2.patch added |
---|
Minor formatting to adhere more closely to the coding standards available at https://www.mythtv.org/wiki/Coding_Standards
comment:10 Changed 3 years ago by
Thank you for the patch. I tried it and it also fixes #12877 which has a link to a sample.
comment:11 Changed 3 years ago by
Status: | infoneeded → assigned |
---|
comment:12 Changed 3 years ago by
I tested using the sample file from #12877. This patch fixes that and does not affect subtitles in other test files I was using.
comment:13 Changed 3 years ago by
Milestone: | unknown → 29.1 |
---|---|
Status: | assigned → accepted |
comment:14 Changed 3 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:16 Changed 3 years ago by
Owner: | changed from Peter Bennett to Peter Bennett |
---|
Having the same problem as described above.
Please find at the following link a 1min, 75mb recording showing the subtitle problem: https://file.town/download/e3ka02xyqoncxn279att8llya
subtitle extraction with mythccextractor, having a lot of missing subtitles. The second shown subtitle for example lasts for 16 seconds, missing 6 other subtitles. https://file.town/uploaded/itqaxntn5ymjm3yh7kqxovsl3
subtitle extraction with ccextractor showing all subtitles correct: https://file.town/uploaded/lxf1t0427pn8mgrm16yxymri5
I am hearing impaired, hoping you could have a look into this issue.