Skip to content

Replace temporary Files with tesseract stdin/stdout#21

Open
jgibbon wants to merge 2 commits intodesmondmorris:masterfrom
jgibbon:stream
Open

Replace temporary Files with tesseract stdin/stdout#21
jgibbon wants to merge 2 commits intodesmondmorris:masterfrom
jgibbon:stream

Conversation

@jgibbon
Copy link
Copy Markdown

@jgibbon jgibbon commented Jan 28, 2015

This change introduces Tesseract.processStream() to simplify usage with build tools and reduce I/O.
Tesseract.process() uses it but – except for an added return value – should remain compatible. (The tests pass, but I did not use it with options.config explicitly.)
I also felt like adding another small test case.

jgibbon added 2 commits January 28, 2015 00:07
introduce Tesseract.processStream() to simplify usage with build tools and reduce I/O
Tests failed on wercker while passing locally, so let's try "stdin stdout" instead of "- -".
@morenoh149
Copy link
Copy Markdown

could you fix the errors here? https://app.wercker.com/#buildstep/54c93225cc09f996373243af

@jgibbon
Copy link
Copy Markdown
Author

jgibbon commented Feb 28, 2015

I couldn't recreate the error locally, with a more or less recent version of tesseract. The version of tesseract on wercker does not seem to accept stdin at all (initially I tried the alternate parameters as you can see), so I'm afraid I won't be able to do anything to fix the errors there... I'm sorry if I wasted your time ;)

@morenoh149
Copy link
Copy Markdown

time to email wrecker :)

@ruudud
Copy link
Copy Markdown

ruudud commented Feb 12, 2016

bump

This would be highly appreciated. 👍

Any help needed in making it work?

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.

3 participants