-
Notifications
You must be signed in to change notification settings - Fork 32
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
PageType.get_AllRegions to list all kinds of regions #479
Conversation
Codecov Report
@@ Coverage Diff @@
## master #479 +/- ##
==========================================
- Coverage 80.10% 80.02% -0.09%
==========================================
Files 40 40
Lines 2388 2393 +5
Branches 442 444 +2
==========================================
+ Hits 1913 1915 +2
- Misses 403 406 +3
Partials 72 72
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
…to get-all-regions
…rder=reading-order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proposal to get recursion (with both finite or arbitrary depth) – not tested!
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## master #479 +/- ##
==========================================
- Coverage 79.71% 79.62% -0.10%
==========================================
Files 41 41
Lines 2490 2493 +3
Branches 477 478 +1
==========================================
Hits 1985 1985
- Misses 417 420 +3
Partials 88 88
Continue to review full report at Codecov.
|
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe CHANGELOG.md needs to be updated. Also, we could try to advertise the new API a little, e.g. in ocrd_models/README.md
(or even up in core/README.md
)...
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
…oupIndexed properly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Splendid! Nothing left to be desired for a new (simpler + more powerful) generation of processors.
Co-authored-by: Robert Sachunsky <38561704+bertsky@users.noreply.github.com>
First step to have a type-agnostic listing of all Regions, Text-, Table-, Graphics- etc.
Currently only available on the
Page
element,ordering by reading order not yet implemeneted.Is now implemented.#240 (comment)
Since we've been adding a few methods to the generateDS code now, I've refactored the code so that the methods can be defined in their own file with proper syntax highlighting etc..