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

Invert subtitle image before the OCR #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MatMaul
Copy link

@MatMaul MatMaul commented Feb 22, 2018

It makes it work way better on my side, especially
with subtitles containing multiple lines.

This currently sits on top of #69, but it can be adapted for master if needed.

It also remove Tesseract 2 support.
It makes it work way better on my side, especially
with subtitles containing multiple lines.
@MatMaul
Copy link
Author

MatMaul commented Feb 28, 2018

It helps a lot with Tesseract v4, with v3 the result is more mitigated but it still seems to help a bit.
Tesseract v4 is globally a lot better than v3 with this patch.
However it is quite slow, hence #69 ;)

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

Successfully merging this pull request may close these issues.

1 participant