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

Align the content model of <additional> with other top-level elements within <msDesc> #2195

Open
schassan opened this issue Oct 12, 2021 · 8 comments

Comments

@schassan
Copy link

schassan commented Oct 12, 2021

The content model of <additional> is much stricter than the ones of the other top-level elements within <msDesc>, except <msIdentifier> (for obvious reasons).

I would like to ask to relax the content model of <additional> and allow <p> and <ab>.

BTW: We need a more relaxed content model in the scope of the Handschriftenportal.

@sydb
Copy link
Member

sydb commented Oct 12, 2021

Hi Torsten! (I suppose I should say “@schassan” :-)
Can you be more precise about what it is you want to tuck into <additional> (that does not fit well into additional/adminInfo/note or additional/surrogates/desc or whatever)?
I do not, off the top of my head, recall why the content model of <additional> is as it is instead of, say { ( adminInfo?, surrogates?, listBibl? ) | ( p | ab )+ } (a content model that I would not be completely happy with, but would perhaps fit the mores of the rest of MS Description somewhat better). Maybe David Birnbaum, Matthew Driscoll, Merrilee Proffitt, or @lb42 can say?

@lb42
Copy link
Member

lb42 commented Oct 12, 2021

Would you be happier if the content model were changed so that <additional> disappeared entirely, and was replaced by ( (adminInfo?, surrogates?, listBibl? ) ? As far as I recall, it was only invented so as to add a bit more structure to this rather miscellanous collection of metadata.

@schassan
Copy link
Author

I wrote a lengthy answer to @sydb and @jamescummings on the list but want to repeat here that we are in need of a container for fulltext of different kinds. @sydb's suggestion would be easier to implement I think while I kind of like @lb42 s suggestion as well. But that might break existing documents and is thus not the favoured solution I guess?

@holfordm
Copy link

@schassan would it be worth sharing some examples of the material you'd like to be able to include? I just wonder if rather than p or ab there might be more semantically rich options that might be better for future interoperability

@larkvi
Copy link

larkvi commented Sep 19, 2022

Per the msDesc SIG meeting at TEI2022, there was general agreement that we have no strong opinions against p in additional and would accordingly support adding it to meet the needs of @schassan's project.

@sydb
Copy link
Member

sydb commented Sep 25, 2022

Which brings us back to my question — would

model.pLike+ | ( adminInfo?, surrogates?, listBibl? )

be sufficient, or would you need

( adminInfo?, surrogates?, listBibl?, model.pLike* )

Things to keep in mind:

  • Most elements in the MS description module that allow model.pLike have a content like the first possibility above: i.e. they allow EITHER a sequence of one or elements from model.pLike OR a sequence of more structured, semantically meaningful elements, BUT NOT both.[1]
  • <additional> is not a repeatable element, so if you want to have both <surrogates> and a paragraph of prose you need the second approach.

Note

[1] I did a quick check, according to which:
The following elements follow this “EITHER paragraphs OR structured semantic stuff” approach: <custodialHist>, <decoDesc>, <handDesc>, <history>, <layoutDesc>, <msContents>, <msDesc>, <msFrag>, <msItem>, <msItemStruct>, <msPart>, <objectDesc>, <recordHist>, <scriptDesc>, <sealDesc>, <supportDesc>, and <typeDesc>.

The following elements allow paragraphs interspersed with other content: <binding>, <bindingDesc>, <physDesc>, and <seal>.

@schassan
Copy link
Author

schassan commented Sep 7, 2023

Sorry for not having commented on that earlier, but this would be sufficient:

model.pLike+ | ( adminInfo?, surrogates?, listBibl? )

@sydb
Copy link
Member

sydb commented Sep 8, 2023

See PR #2470.

ebeshero added a commit that referenced this issue Nov 10, 2023
Impliment #2195 per request TEI msDesc SIG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants