CEA-708 sentence capitalization #189

New Issue

claunia · 2026-01-29T16:37:28Z

claunia commented

2026-01-29 16:37:28 +00:00

Originally created by @cactusGit on GitHub (Sep 21, 2016).

It's missing. Looks like clever_capitalize and correct_case_with_dictionary used only for other subs. I can make pull request with mostly copy-pasted code, but you could make it more architecturally correct.

Originally created by @cactusGit on GitHub (Sep 21, 2016). It's missing. Looks like `clever_capitalize` and `correct_case_with_dictionary` used only for other subs. I can make pull request with mostly copy-pasted code, but you could make it more architecturally correct.

claunia closed this issue

2026-01-29 16:37:29 +00:00

claunia commented

2026-01-29 16:37:29 +00:00

@cfsmp3 commented on GitHub (Sep 21, 2016):

Pending for now. In theory subtitles from 708 should be proper cased unless someone is really lazy.

I prefer to wait until 708 is stable (meaning subs extraction is good and we get to the point in which we prefer 708 over 608 every time) before adding features/bloat that makes code harder to debug.

@cfsmp3 commented on GitHub (Sep 21, 2016): Pending for now. In theory subtitles from 708 should be proper cased unless someone is really lazy. I prefer to wait until 708 is stable (meaning subs extraction is good and we get to the point in which we prefer 708 over 608 every time) before adding features/bloat that makes code harder to debug.

claunia commented

2026-01-29 16:37:30 +00:00

@cfsmp3 commented on GitHub (Jan 20, 2017):

Closing, since it's not going to happen - it just doesn't seem needed for 708, I've been looking at lots of samples lately.

@cfsmp3 commented on GitHub (Jan 20, 2017): Closing, since it's not going to happen - it just doesn't seem needed for 708, I've been looking at lots of samples lately.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#189