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

EDMG BNF #2732

Merged
merged 4 commits into from
Dec 16, 2024
Merged

EDMG BNF #2732

merged 4 commits into from
Dec 16, 2024

Conversation

DenisNosnitsin1970
Copy link
Contributor

Minor changes in mss, stub for Griaule 32.

Minor changes in mss, stub for Griaule 32.
<material key="parchment"/>
</support>
<extent>
<measure unit="leaf" quantity="127">27</measure>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Number in quantity should match the number in the tag.

Is it by accident the same value as in DHAS024 above, or possibly a Copy&Paste error?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a mistake.

<textLang mainLang="gez"/>

<notatedMusic>
<desc>Musical notation in red and black thoughout the text.</desc>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

throughout

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

<incipit xml:lang="gez">ወዛካርያስ᎓ ወፈጸምያነ᎓ ትንቢት᎓ ከዊኖሙ᎓ ወዳንኤልሰ᎓ ወሕዝቅኤልሰ᎓ በፄዋዌ᎓ ተነበዩ᎓ በቢሎን᎓
ወኮኑ᎓ ካህናተ᎓ ኵሎሙ᎓ <hi rend="rubric">፯፡</hi>
<explicit xml:lang="gez">ቀዳሚ፡ ኵሉ፡ ነግሠ፡ ሳኦል፡ ለእስራኤል፡ ዓመታት፡ ፵ዘእም፡ ነገደ፡ ብንያም፡ ወእምነገደ፡ ይሁዳ፨ ወእምድኅረዝ፡ ነግሠ፡ ዳዊት፡</explicit>
<incipit xml:lang="gez">ወዛካርያስ፡ ወፈጸምያነ፡ ትንቢት፡ ከዊኖሙ፡ ወዳንኤልሰ፡ ወሕዝቅኤልሰ፡ በፄዋዌ፡ ተነበዩ፡ በቢሎን፡ ወኮኑ፡ ካህናተ፡ ኵሎሙ፡ <hi rend="rubric">፯፡</hi>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you place the incipit after the explicit? Will this be presented on the website in the same order or is the order always, that the incipit is placed before the explicit after it is transformed, as it should be?

<msIdentifier>
<repository ref="INS1017EndaMaryamDengelat"/>
<idno>EDMG-045</idno>
<idno facs="EDMG/045/EDMG-045_" n="58">EDMG-045</idno>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not have these images I believe

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are coming, need some processing.

<msDesc xml:id="ms">
<msIdentifier>
<repository ref="INS1017EndaMaryamDengelat"/>
<idno>EDMG-045</idno>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why two idno?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed.

<repository ref="INS1017EndaMaryamDengelat"/>
<idno>EDMG-045</idno>
<idno facs="EDMG/045/EDMG-045_" n="58">EDMG-045</idno>
</msIdentifier>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please also check EDMG001 EDMG002 etc that have multiple idno elements - there should only be one, all others should be in altIdentifier

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed.

<textLang mainLang="gez"/>
<note>Only two thirds of the text leaf, originally belonging probably to
Ms <ref type="mss" corresp="DHAS023">Dabra Hāllelo ʾƎndā ʾAbuna Sāmuʾel DHAS-023</ref>.</note>
</msItem>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need a relation to show this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed.

<collection>Manuscrits orientaux</collection>
<collection>Fonds éthiopien</collection>
<collection>Griaule</collection>
<idno>BnF Éthiopien 336 (Griaule 32)</idno>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

                        <idno>BnF Éthiopien 336</idno>
                        <altIdentifier>
                            <idno>Griaule 32</idno>
                        </altIdentifier>

you do not need Griaule 32 twice

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is true. Here I followed "the tradition", the <idno>BnF Éthiopien 336 (Griaule 32)</idno> was designed to be displayed as "BnF Éthiopien 336 " in the app, as other Griaule mss, with double shelf mark (thorough numeration and the numeration according to "the fond"). I think it has its reasons, and it is correct to display them both. If I place Griaule 32 in <altIdentifier/> only, it is now displayed in small letters on the right side, separate from the shelf mark, not very well visible. This is ok for me, but will bring some inconvinience for the user (usually Griaule mss are referred without BnF shelfmarks). Should I change it? (I can change also other Griaule records).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should always be very careful to keep encoding and visualization apart.
Each identifier and each shelfmark MUST be encoded separately.
If for some reason the visualization is not what you want then you need an issue for visualization. We can always have a different way to show altIdentifiers, or we can have separate rules for different types, this is all possible, but encoding has nothing to do with visualization.

Copy link
Contributor

@nafisa-valieva nafisa-valieva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Looks good for me!

Updates, corrections of shelfmarks.
@DenisNosnitsin1970 DenisNosnitsin1970 merged commit 132b601 into master Dec 16, 2024
1 check passed
@DenisNosnitsin1970 DenisNosnitsin1970 deleted the EDMGtheRest branch December 16, 2024 21:02
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

Successfully merging this pull request may close these issues.

4 participants