[BUG] Negative -delay values produce empty .srt output #669

Closed
opened 2026-01-29 16:50:35 +00:00 by claunia · 5 comments
Owner

Originally created by @muzicman0 on GitHub (Nov 7, 2021).

CCExtractor version: 0.93

Necessary information

  • Is this a regression (i.e. did it work before)? YES (worked 0.88, does not work on 0.93)
  • What platform did you use? Windows
  • What were the used arguments? -delay -10000

Video links

https://1drv.ms/u/s!AmIBKnKiJ1bih7QqG_PJfxvirnnV5g?e=bXHJe2

link above, but any of my recorded TV files have the same issue.

Additional information

Negative -delay option on (at least) the CLI doesn't work (empty .srt file). It did work on version 0.88. I recently updated to the latest version (0.93) and it no longer works. tried multiple files, and multiple negative values.

Positive delay options work fine (IE: -delay 1000)

Originally created by @muzicman0 on GitHub (Nov 7, 2021). CCExtractor version: 0.93 # Necessary information - Is this a regression (i.e. did it work before)? YES (worked 0.88, does not work on 0.93) - What platform did you use? Windows - What were the used arguments? `-delay -10000` # Video links https://1drv.ms/u/s!AmIBKnKiJ1bih7QqG_PJfxvirnnV5g?e=bXHJe2 link above, but any of my recorded TV files have the same issue. # Additional information Negative -delay option on (at least) the CLI doesn't work (empty .srt file). It did work on version 0.88. I recently updated to the latest version (0.93) and it no longer works. tried multiple files, and multiple negative values. Positive delay options work fine (IE: -delay 1000)
Author
Owner

@PunitLodha commented on GitHub (Nov 19, 2021):

This was fixed recently. You can try compiling the code from the master branch

@PunitLodha commented on GitHub (Nov 19, 2021): This was fixed recently. You can try compiling the code from the master branch
Author
Owner

@muzicman0 commented on GitHub (Dec 1, 2021):

Thanks. I saw that it had been fixed. Compiling is probably beyond me right now, so I will just wait for now.

@muzicman0 commented on GitHub (Dec 1, 2021): Thanks. I saw that it had been fixed. Compiling is probably beyond me right now, so I will just wait for now.
Author
Owner

@muzicman0 commented on GitHub (Dec 16, 2021):

.94 is up, so I am checking it out to make sure it works for me. Thanks!

@muzicman0 commented on GitHub (Dec 16, 2021): .94 is up, so I am checking it out to make sure it works for me. Thanks!
Author
Owner

@muzicman0 commented on GitHub (Dec 16, 2021):

I guess I will just stick with the old version. .94 doesn't work on some files at all, and only pulls a few minutes of captions on others.

@muzicman0 commented on GitHub (Dec 16, 2021): I guess I will just stick with the old version. .94 doesn't work on some files at all, and only pulls a few minutes of captions on others.
Author
Owner

@PunitLodha commented on GitHub (Dec 17, 2021):

This could be a bug. Could you create a new issue for it? With details about which files aren't working.

Also, there was a change in the flags. In 0.94 :-

  • if no params are provided:- extract whatever is available, if both(708 and 608) are available then extract both
  • If -1/-2 is provided:- extract only 608 subs
  • if -svc is provided:- extract only 708 subs
  • if both -1/-2 and -svc are provided:- extract both 608 and 708
@PunitLodha commented on GitHub (Dec 17, 2021): This could be a bug. Could you create a new issue for it? With details about which files aren't working. Also, there was a change in the flags. In 0.94 :- - if no params are provided:- extract whatever is available, if both(708 and 608) are available then extract both - If -1/-2 is provided:- extract only 608 subs - if -svc is provided:- extract only 708 subs - if both -1/-2 and -svc are provided:- extract both 608 and 708
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#669