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

Man page description of configs and parameters #2220

Merged
merged 1 commit into from
Feb 16, 2019

Conversation

cjmayo
Copy link
Contributor

@cjmayo cjmayo commented Feb 3, 2019

Three commits here, first two should be straightforward:

  • get.image is mentioned in tesseract(1), as get.images and not installed
  • A typo and missing full stops in tesseract(1)

At first read of tesseract(1) I didn't find it easy to follow what a config was and the third commit is my attempt to improve that. That's more subjective so please do look at it critically.

Copy link
Collaborator

@Shreeshrii Shreeshrii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Description for lstm.train should be

lstm.train - Output files used by LSTM training ('outputbase'.lstmf).

@cjmayo
Copy link
Contributor Author

cjmayo commented Feb 4, 2019

Updated lstm.train as suggested.

@stweil
Copy link
Member

stweil commented Feb 5, 2019

It would be easier to have separate pull requests instead of one with several unrelated commits. The 2nd commit is fine for me, but the others are more controversial or need more care. I wonder for example whether renaming the current file tessdata/configs/get.image to tessdata/configs/get.images would be a better solution.

@cjmayo
Copy link
Contributor Author

cjmayo commented Feb 5, 2019

OK. Created:

Rename get.image config to get.images and install #2223
Tidy tesseract(1) #2224

I'll rebase this one if and when the others are merged because there is a slight overlap.

Try to make the relationship between configs, -c and --print-parameters
clearer by always using parameter and not variable.

Include the filenames created by each config.
@cjmayo
Copy link
Contributor Author

cjmayo commented Feb 6, 2019

Rebased on master, including restoring reference to get.images.

@zdenop zdenop merged commit 48be357 into tesseract-ocr:master Feb 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants