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

obsolete or odd markup in files with textparts #171

Open
emylonas opened this issue May 12, 2021 · 1 comment
Open

obsolete or odd markup in files with textparts #171

emylonas opened this issue May 12, 2021 · 1 comment
Assignees

Comments

@emylonas
Copy link
Contributor

caes0418: The diplomatic transcription has the type "obverse" for all the fragments. Would "fragment" be clearer? Alternatively, the edited transcription uses <app>. The commentary refers to fragments 11a and 11c. Could we use those as values for @subtype?

caes0501: monogram - do we do them as <choice> or <expan>? or is the <choice> correct?

caes0502 - has a text part, but only one. Should it even be there? Same monogram question as above.

More monograms: caes0503, caes0504 (this indicates that the text is on the reverse. But no text on obverse. Not sure it's a textpart, but not deal breaker, caes0507, caes0510 (1 textpart again), caes0511, caes0512, caes0513

caes0516, caes0517 - only one textpart

zeichman added a commit that referenced this issue May 13, 2021
First go at a few problems raised in #171 and a few inscriptions that Nesya fixed as well.
@zeichman
Copy link
Collaborator

One problem with caes0418 is that <supplied> tags are not presently allowed within <unclear> tags. This does not account for the <choice><unclear>a<supplied>bcd</supplied></unclear><unclear>a<supplied>lphabet</supplied></unclear></choice> situation. I think you're right about tagging it as fragments, though.

Regarding monograms - the <figure> tag is allowed in <expan> - I'm also not sure whether the monogram would be within <am> or <abbr>. I feel like it should be the former (cf. L for ἔτους).

I have removed the textpart for caes0516 and caes0517, as it doesn't really serve a purpose; that information makes more sense in the commentary, in these cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants