Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] X-TIMESTAMP-MAP doesn't show up with DVB subs #1166

Open
7 tasks done
NilsIrl opened this issue Jan 1, 2020 · 5 comments
Open
7 tasks done

[BUG] X-TIMESTAMP-MAP doesn't show up with DVB subs #1166

NilsIrl opened this issue Jan 1, 2020 · 5 comments

Comments

@NilsIrl
Copy link
Contributor

NilsIrl commented Jan 1, 2020

CCExtractor version: 0.88

  • I have read and understood the contributors guide.
  • I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
  • I have checked that the issue I'm posting isn't already reported.
  • I have checked that the issue I'm porting isn't already solved and no duplicates exist in closed issues and in opened issues
  • I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.
  • I have used the latest available version of CCExtractor to verify this issue exists.

My familiarity with the project is as follows (check one, eg [X] - and delete unchecked ones):

  • I have used CCExtractor just a couple of times.

Necessary information

  • Is this a regression (did it work before)? idk (probably not)
  • What platform did you use? [X] Linux

arguments:

./ccextractor -datapid 0xc37 -out=webvtt files/test1_dvbsub.ts

Video link: http://akini.mbnet.fi/videos/test1_dvbsub.ts

Related to #1127 in goal though they have different "symptoms" and arise in different cases.

@NilsIrl
Copy link
Contributor Author

NilsIrl commented Jan 1, 2020

Info for some who wants to fix it:

This is the line that calls the function that adds the header (as well as the subtitles) when the input file isn't dvb:

wrote_something = write_cc_buffer_as_webvtt(data, context);

This is the line that calls the function that doesn't do that:

wrote_something = write_cc_bitmap_as_webvtt(sub, context);

So you basically want to second line to the the same thing as the first line. Feel free to ask more information if my instructions are unclear.

@NilsIrl
Copy link
Contributor Author

NilsIrl commented Jan 1, 2020

This is the line in write_cc_buffer_as_webvtt that adds the header:

write_webvtt_header(context);

You want this line to be called in write_cc_bitmap_as_webvtt

@karansdoshi
Copy link

Hello I am a newbie and I'm interested in solving this issue.Both the functions have the line write_webvtt_header(context) with the same context.What exactly is the issue?Can you elaborate about buffer and bitmap at bit.
tess.log

test1_dvbsub.zip
This TS file has more than one program. These are the program numbers found:
1501
1502
1503
fin.traineddata not found! Switching to English
VBI/teletext stream ID 5100 (0x13ec) for SID 1503 (0x5df)
Premature end of file - Transport Stream packet is incomplete (expected 188 bytes, got 92).
100% | 03:41
Number of NAL_type_7: 0
Number of VCL_HRD: 0
Number of NAL HRD: 0
Number of jump-in-frames: 0
Number of num_unexpected_sei_length: 0

Min PTS: 21:54:50:849
Max PTS: 21:58:32:650
Length: 00:03:41:801
Done, processing time = 35 seconds
I am not getting the reason behind premature end.Thanks

@NilsIrl
Copy link
Contributor Author

NilsIrl commented Jan 22, 2020

Looking back at the code, I think this might be solved by #1137 (if it ever gets merged).

@anshravalll
Copy link

I believe this issue has been resolved due to the introduction of #1464. Anybody can easily attach the X-TIMESTAMP-MAP by utilizing the --timestamp-map option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants