-
Notifications
You must be signed in to change notification settings - Fork 52
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
Inspect teiHeder elements that contain text content #215
Comments
Element ParlaMint/Schema/ParlaMint-teiCorpus.rng Lines 332 to 341 in 4a53f9b
By removing such cases, we lose information in parliament language. Do we want to have such info? |
Well, if we are removing the text content in birth and death (which can be formatted as per object language), then we should also remove sex. So, yes, let us have a consistent policy: if metadata info is available in attribute value, then we do not have it in the text content. |
I think we now have a complete list, it is actually shorter than I thought. Here are some further comments:
@matyaskopp, do you agree? And can you implement fixes in processing, and I can do the schema + ODD. |
@TomazErjavec Agree. |
Out of time now but will do it soon. Note that these are massive changes, so there will be lots of errors! |
Thinking about this some more, I think we should allow text content for |
OK, did it, I think: Schema corrected in devel, TEI in description and description-desc. Also merged description into description-desc, was a nightmare, I hope I haven't messed things up. There are of course hundreds of errors now in |
List of metadata elements that contain text content only.
(related to #183 , #205 (comment))
The final list is in this comment. The discussion is below in comments if necessary.
remove text content from
Note that the
@xml:lang
attribute from the above elements should also be removed.preserve text content
The text was updated successfully, but these errors were encountered: