Skip to content
This repository was archived by the owner on Apr 27, 2026. It is now read-only.

Output UTF8-HTML-files by default and show args.minscale when skipping args #312

Open
NormanTUD wants to merge 3 commits intoocropus-archive:masterfrom
NormanTUD:master
Open

Output UTF8-HTML-files by default and show args.minscale when skipping args #312
NormanTUD wants to merge 3 commits intoocropus-archive:masterfrom
NormanTUD:master

Conversation

@NormanTUD
Copy link
Copy Markdown

No description provided.

@zuphilip
Copy link
Copy Markdown
Collaborator

zuphilip commented Sep 1, 2018

@NormanTUD Looks okay in general. How about the overview page in ocropus-visualize-results, add the UTF8 part there as well?

@NormanTUD
Copy link
Copy Markdown
Author

Also use UTF8-Output for ocropus-visualize-results

@NormanTUD NormanTUD closed this Sep 6, 2018
@NormanTUD NormanTUD reopened this Sep 6, 2018
@zuphilip
Copy link
Copy Markdown
Collaborator

Ah, sorry for the delay. I think here before this block you have to include a line about UTF8 as well: https://github.com/tmbdev/ocropy/blob/65b14307f1dbd3b55006457f6babebdf20cdde9d/ocropus-visualize-results#L102-L107

Copy link
Copy Markdown
Collaborator

@kba kba left a comment

Choose a reason for hiding this comment

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

Fine by me, more for the fix in gpageseg than the UTF-8 header (which is still invalid HTML since body tags are missing).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants