Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
structMap[@TYPE=OCR-D-LOGICAL] / FULLDOWNLOAD #154
structMap[@TYPE=OCR-D-LOGICAL] / FULLDOWNLOAD #154
Changes from 12 commits
d8131e9
27113c3
cae8a5c
3a4d7b2
e14f6af
455da44
cbdcf42
4f56d49
d4d7fa5
197977e
d65acfc
c8d6182
db50238
9b1dd7d
02b6a7a
0291904
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
It does not make sense to put this section past
Grouping files by page
– the latter should be integrated into the former as a subsection!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.
here my proposal for a new document structure:
Requirements on handling METS/PAGE
Metadata
1.1 Unique
@ID
for the document processedImages
2.1. Pixel density of images must be explicit and high enough
2.2. No multi-page images
2.3 Image coordinates
2.4 If in PAGE then in METS
File Group
mets:fileGrp
3.1
@USE
syntaxExamples
File
mets:file
4.1
@ID
syntaxExamples
4.2
@MIMETYPE
syntaxExamples
Examples (Media Type for PAGE XML)
Grouping files by page
mets:structMap
Example
5.1
@TYPE
syntaxExample
Range of pages
mets:structLink
Example
Paths
7.1 Always use URL or relative filenames
Example
Recording processing information in METS
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.
In that case, I would:
But I wonder: where in that outline did
Fulldownload
go? Is it still subsumed under 4.1 for you? (We discussed this elsewhere: then you cannot make these subsections self-contained.)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.
That's right, I think 2.3 is better found in page.md.
Yes,
Fulldownload
is a section/part under 4.1There 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.
Requirements on handling METS/PAGE
1 Metadata
1.1 Recording processing information in METS
1.2 Unique
@ID
for the document processed2 Images
2.1. Pixel density of images must be explicit and high enough
2.2. No multi-page images
2.3 If in PAGE then in METS
3 File Group
mets:fileGrp
3.1
@USE
syntaxExamples
3.2
@USE="FULLDOWNLOAD_..."
Examples
4 File
mets:file
4.1
@ID
syntaxExamples
4.2
@MIMETYPE
syntaxExamples
Examples (Media Type for PAGE XML)
5 Grouping files by page
mets:structMap
Example
5.1
@TYPE
syntaxExample
6 Range of pages
mets:structLink
Example
7 Paths
7.1 Always use URL or relative filenames
Example
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.
Is this resolved?
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.
No, AFAICS it is not. The section
Fulldownload
should still be part of the file ID syntax (differentiating between page-local and document-global naming scheme, but not trying to formulate this "self-contained"). Also, the section aboutgrouping files by structMap
should come below fileGrp and file ID sections.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.
(Or did you want to do all that in a separate PR, or just wait for the merge with master?)
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.
Yes!