Maria-Edgeworth-Letters-Project / me-tei

Repository holding TEI XML and documentation for MELP.
Other
1 stars 4 forks source link

Editorial note #37

Closed mlhale7 closed 7 months ago

mlhale7 commented 7 months ago

What does this Pull Request do?

This PR addresses the requested changes outlined in issue #34

How should this be tested?

Ensure that the change made to the text within editorialDecl matches expectations and that this change has been made across all XML files.

Additional changes had to be made to make all existing XML files validate. Please take a special look at Edgeworth_Bodleian_696_1819_14_17.xml. This letter had two figureDesc tags that were moved by me and might potentially not be situated as needed. I've attached the scans below. We can potentially discuss this more in our RA meeting instead and make changes after this PR is closed. Edgeworth_Bodleian_696_1819-04-17_page06 Edgeworth_Bodleian_696_1819-04-17_page07

Additional notes

Try to avoid copying and pasting the ref values for persName and placeName. There were several instances in which a ref value went to the Personography, but a place was referenced. There are likely more that I didn't catch since this mistake doesn't cause a validation error. Please also always remember to add a file extension (.xml).

hilaryhavens commented 7 months ago

Thanks for flagging this, Meredith! In both cases, I would move the

, and there was also some missing text. I just want to confirm [<figure>](https://www.tei-c.org/release/doc/tei-p5-doc/en/html/ref-figure.html) should only enclose the figure description and not the text related to it so we know precisely where to place it? On that assumption, I have moved also the closing
tags.

#1

aldermen<lb/>
               glyphic <hi rend="underline">frights </hi> <figure></figDesc>three figures, in a kneeling posture,  drawn facing away from a figure of Osiris</figDesc> all pointing with<lb/>

Should be:

aldermen <figure></figDesc>two figures, in a kneeling posture,  drawn facing away from a figure of Osiris, with two flourishes beneath</figDesc></figure> embossed gold <choice><orig>hiero<lb/>
glyphic</orig><reg>hieroglyphic</reg></choice> <hi rend="underline">frights</hi> all pointing with<lb/>

#2

a heavy stair <figure><figDesc>a woman holding up twelve candle branches with a small square base</figDesc>case while her feet stand on a globe so<lb/>
 small that it never could support her &amp; it seems al-<lb/>
 ways stepping from under her - A candalabra figure<lb/>
<add>in the dining room</add><lb/>
with arms painfully holding 12 candle branches<lb/>
heavier &amp; larger than herself and without a resting<lb/>
 place for her poor feet - on the contrary with<lb/>
 a cruel gold basket hung on her ancles ! - </figure><lb/>

Should be:

a heavy stair case while her feet stand on a globe so<lb/>
small that it never could support her &amp; it seems <choice><orig>al-<lb/>
ways</orig><reg>always</reg></choice> stepping from under her - <add>also a slice of a bronze</add><del>A</del> candalabra figure<lb/>
<add>in the dining room</add><lb/>
with <figure><figDesc>a woman holding up twelve candle branches with a small square base</figDesc></figure> arms painfully holding 12 candle branches<lb/>
heavier &amp; larger than herself and without a resting<lb/>

Thanks and will look forward to discussing this further on Thursday, if necessary!

mlhale7 commented 7 months ago

@hilaryhavens - I think it may make sense to simply focus on the purpose of the PR and address this on Thursday. I had to change the file to make it validate so that I could run XSLT on it, but it's really beyond the scope of this PR and shouldn't hold everyone else up for a few days. The problem is that </figure> can't be placed after a </lb> and still have the file validate. Typically figure seems to be being used in TEI for an added image file versus something in-line with text. Honestly, I think I need to look at this more.

hilaryhavens commented 7 months ago

Thanks, Meredith! We can talk about where precisely the <figure> tag is placed in each case on Thursday, but this is related to my first point - that we should be doing <figure><figDesc></figDesc></figure> instead of <figure><figDesc></figDesc> some text related to the image that may span multiple lines</figure> . Does this answer the question? Thanks!

mlhale7 commented 7 months ago

@hilaryhavens - yes, that's how I've changed the tags in the commit I've added. I don't think it's possible to note effectively where the figure interrupts text as was initially attempted.

elizawilcox commented 7 months ago

@mlhale7 @hilaryhavens I initially attempted (as Meredith noted) to try and capture the text interruptions, but if that's not possible, I can make edits to the workflow after Thursday's meeting when we discuss to note that the priority for describing figures is description, not necessarily total replication since a

can't go across tags.

mlhale7 commented 7 months ago

People are welcome to look closer at the changes I've made to that single file here - Encoded Letters/Edgeworth_Bodleian_696_1819_04_17.xml

Otherwise I'd appreciate feedback on whether issue #34 has been fully addressed. If that issue has been addressed, please approve. If it hasn't, please comment.

rrunia commented 7 months ago

Hi all, I agree with Hilary's suggestions. Cheers, Robin

Robin Runia, Ph.D. Professor of English Kellogg Endowed Professorship in English Xavier University of Louisiana 1 Drexel Drive, Box 89 New Orleans, LA 70125

@.*** she/her/hers

Faculty Editor, XULANEXUShttps://digitalcommons.xula.edu/xulanexus/ Series Editor, https://udpress.udel.edu/book-series/early-modern-feminisms/ Early Modern Feminisms, https://udpress.udel.edu/book-series/early-modern-feminisms/ University of Delaware Presshttps://udpress.udel.edu/book-series/early-modern-feminisms/ Editor, Maria Edgeworth Letters Projecthttps://mariaedgeworth.org/


From: hilaryhavens @.> Sent: Monday, March 18, 2024 9:32 AM To: Maria-Edgeworth-Letters-Project/me-tei @.> Cc: Robin Runia @.>; Review requested @.> Subject: Re: [Maria-Edgeworth-Letters-Project/me-tei] Editorial note (PR #37)

@hilaryhavens approved this pull request.

This looks great - thanks for adding in the paragraph about ME grammar / spelling!

— Reply to this email directly, view it on GitHubhttps://github.com/Maria-Edgeworth-Letters-Project/me-tei/pull/37#pullrequestreview-1943263455, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMWT2WCJTABOIL7YRCDMCP3YY33H7AVCNFSM6AAAAABE3PPGHSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNBTGI3DGNBVGU. You are receiving this because your review was requested.Message ID: @.***>