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

AlternativeImage "comments" attribute for grayscale images #41

Closed
kba opened this issue May 8, 2018 · 7 comments
Closed

AlternativeImage "comments" attribute for grayscale images #41

kba opened this issue May 8, 2018 · 7 comments
Assignees
Milestone

Comments

@kba
Copy link
Member

kba commented May 8, 2018

Such as ocropus' nrm. For other transformations we use the form binarized, dewarped etc (c.f. #29). How do we name alternative images that are kind-of-binarized but with some gray at the edges of glyphs?

@wrznr
Copy link
Contributor

wrznr commented May 9, 2018

I have to check the idea and the "techniques" of this image transformation process. But given the file extension, I opt for normalized (with further explanations in the glossary).

@kba
Copy link
Member Author

kba commented May 9, 2018

The reasoning why ocropy does it like this is explained in the ocropy wiki.

normalized makes sense esp. considering the nrm suffix but is a very generic term. semi-binarized? grayscalized? quasi-binarized?

@wrznr
Copy link
Contributor

wrznr commented May 9, 2018

grayscalized is technically not correct, right? What about ternarized?

@wrznr
Copy link
Contributor

wrznr commented May 9, 2018

It will be grayscale_normalized.

@kba
Copy link
Member Author

kba commented May 14, 2018

OK, so grayscale_normalized for comments of the resulting mets:file. Should we also extend the step property in ocrd-tool.json to contain preprocessing/optimization/grayscale_normalization in addition to preprocessing/optimization/binarization?

@wrznr
Copy link
Contributor

wrznr commented May 14, 2018 via email

@kba kba added this to the 1.1.5 milestone May 15, 2018
kba added a commit to kba/spec that referenced this issue May 15, 2018
@kba
Copy link
Member Author

kba commented May 16, 2018

Is in v1.1.5, both the docs and schema.

@kba kba closed this as completed May 16, 2018
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

No branches or pull requests

2 participants