Skip to content

Enhancement docker build & words and bounds results #49

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Enhancement docker build & words and bounds results #49

wants to merge 7 commits into from

Conversation

levinunnink
Copy link

@levinunnink levinunnink commented Jan 2, 2019 •
edited
Loading

This PR adds three features:

  1. Dockerization of the build process for Tesseract so new versions can be built locally without having to create an EC2 instance for building.
  2. The ability to extract words and their coordinates from images with the getWordsAndBounds function.
  3. Support for using streams in addition to file paths.

@levinunnink levinunnink changed the title Enhancement object bounds & docker build Enhancement docker build Jan 2, 2019
@levinunnink levinunnink changed the title Enhancement docker build Enhancement docker build & words and bounds results Jan 2, 2019
vladholubiev
vladholubiev reviewed Jan 3, 2019
View reviewed changes
vladholubiev
vladholubiev reviewed Jan 3, 2019
View reviewed changes
vladholubiev
vladholubiev reviewed Jan 3, 2019
View reviewed changes
@vladholubiev
Copy link
Member

Thanks a lot for such a feature-heavy contribution @levinunnink 🙌🏻

@randombluff
Copy link

@levinunnink plz merge so no conflicts occur

@levinunnink
Copy link
Author

@maaraanas I don't have permission. Looks like @vladgolubev has to merge this PR.

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

Reviewers

@vladholubiev vladholubiev vladholubiev left review comments

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@levinunnink @vladholubiev @randombluff