Giters

CCExtractor / ccextractor

CCExtractor - Official version maintained by the core team

Home Page:https://www.ccextractor.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[BUG] 608/708 captions not being converted to webvtt

shaunflagg opened this issue · comments

commented

CCExtractor version: 0.94

In raising this issue, I confirm the following:

  • 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.
  • I have ticked all the boxes in this section and to prove it I'm deleting the section completely to remove boilerplate text.

Necessary information

  • Is this a regression (i.e. did it work before)? YES
  • What platform did you use? Linux and Mac
  • What were the used arguments? ccextractor file.mp4 -out-webvtt

Video links

N/A

Additional information

I have some video files with 608/708 embedded captions and have been using ccxtractor to generate webvtt captions files. This has been successful with version 0.88 for some time. I recently installed version 0.94 and get this error

[CEA-708] dtvcc_handle_G0: Window has to be defined first
[CEA-708] dtvcc_handle_CWx_SetCurrentWindow: window [0] is not defined
Segmentation fault
commented

Please share the videos for which you get this error. You can upload them, and share the link here

commented

I am able to extract correct vtt captions for this file on 0.94. Could you please share the full output of ccextractor --version and also tell where have you installed ccextractor from?

commented

Closing because I can't reproduce. I did fix an unrelated issue (small memory leak) with this sample though.

@shaunflagg feel free to reopen if you can reproduce with current master.