Make small fixes to event and eventName #2466

joeytakeda · 2023-09-05T12:55:11Z

From Paderborn F2F: We've now merged @skurzinz wonderful work (#2427) into a new branch (https://github.com/TEIC/TEI/tree/issue_2382_eventName) so that we can easily make little adjustments (rather than dipping into @skurzinz 's fork ) before moving into dev.

In council discussion, we decided to make the following adjustments:

Update events content model to allow idno as a first child (to preserve backwards compatibility) and place eventName in exclusive alternation with model.labelLike and model.pLike
Reorder classes for consistency
Change English desc of event to "contains data relating to anything of significance that happens in time."
Update all other descs to match

The text was updated successfully, but these errors were encountered:

skurzinz · 2023-09-05T13:27:17Z

contains data relating to anything of significance that happens in time.

@joeytakeda could you do the change for German in this branch so that I don’t have to git checkout various upstream changes? @martinascholger could you doublecheck my prose?

<desc versionDate="2023-09-05" xml:lang="de">enthält Daten mit Bezug zu etwas Bemerkenswertem, das in der Zeit geschieht.</desc>

joeytakeda · 2023-09-05T13:35:34Z

Okay I have now done all of what I (and my monolingualism) can do :-)

But in working spending a bit more time with this, I have a few questions (which have to do with event and not eventName and thus may merit consideration only after eventName is merged or in coordination with #367 )

The //exemplum[@xml:lang='de-AT'] is excellent, but some thoughts:
- Is it acceptable to edit this example slightly for the sake of brevity/readability? I would suggest removing the hash @ref on the individual sub-events (i.e. <event ref="#top_1_1_1_1">) since that example already shows the use of @ref on event and there is no entity top_1_1_1_1 defined in this file?
- Each of the events have a @where pointing to "Wien", but, presumably, it should be #Wien (or plc:Wien or some such). Should we append a listPlace/place[@xml:id = 'Wien'] below the <listPerson> in that example?
- None of the persons in the event/listPerson are explicitly referenced in the example, so I would suggest that either we add references to these persons in this example or we simply remove it from the example
Should the order of entities and their lists matter? In this proposal, event can now have many different kinds of entities as children:

   <alternate minOccurs="0" maxOccurs="unbounded">
        <elementRef key="event" minOccurs="1" maxOccurs="1"/>
        <elementRef key="listEvent" minOccurs="1" maxOccurs="1"/>
      </alternate>
      <alternate minOccurs="0" maxOccurs="unbounded">
        <classRef key="model.personLike" minOccurs="1" maxOccurs="1"/>
        <elementRef key="listPerson" minOccurs="1" maxOccurs="1"/>
      </alternate>
      <alternate minOccurs="0" maxOccurs="unbounded">
        <classRef key="model.placeLike" minOccurs="1" maxOccurs="1"/>
        <elementRef key="listPlace" minOccurs="1" maxOccurs="1"/>
      </alternate>
      <classRef key="model.objectLike" minOccurs="0" maxOccurs="unbounded"/>
      <alternate minOccurs="0" maxOccurs="unbounded">
        <elementRef key="relation" minOccurs="1" maxOccurs="1"/>
        <elementRef key="listRelation" minOccurs="1" maxOccurs="1"/>
      </alternate>

Personally, I don't think it should matter in what order you put any of these things (in the current guidelines, the only one of these entities that event allows is event, which must come last).

skurzinz · 2023-09-05T14:31:46Z

The //exemplum[@xml:lang='de-AT'] is excellent, but some thoughts:

Thanks for those. I fully agree, and thus came up with the following snippet, which I would ask you again to include in your branch to avoid git hassle:

<listEvent type="generated">
        <event xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001" ref="https://mrp.oeaw.ac.at/pages/show.html?directory=editions&amp;document=MRP-1-1-01-0-18480401-P-0001.xml" 
          type="session">
          <head>Nr. 1 Ministerrat, Wien, 1. April 1848</head>
          <label>Ministerratssitzung <date when="1848-04-01">1848-04-01</date>, aus Band 1 1/1, März 1848–21. November 1848</label>
          <listEvent type="agenda_items">
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_1">
              <label>I. Enthebung Baron Karl Friedrich v. Kübecks als Finanzminister und Ernennung des Freiherrn Philipp v. Krauß zu seinem Nachfolger</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_2">
              <label>II. Ernennung FML. Peter Zaninis zum Kriegsminister</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_3">
              <label>III. Öffentliche Erklärung über die konstitutionellen Grundsätze; Umformung des Staatsrates</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_4">
              <label>IV. Kriegserklärung Sardiniens; Vorgehen Radetzkys; Bestellung eines Hofkommissärs für Lombardo-Venetien</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_5">
              <label>V. Beeidigung der Minister</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_6">
              <label>VI. Tumulte im Kärntnertortheater</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_7">
              <label>VII. Bewegung der tschechischen demokratischen Partei; Berufung Paul Josef Safaiřiks nach Wien</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_8">
              <label>VIII. Behandlung der aus Lombardo-Venetien vertriebenen Justiz- und politischen Beamten</label>
            </event>
            <event type="agenda_item" xml:id="mrp-events-MRP-1-1-01-0-18480401-P-0001-top_1_1_1_9">
              <label>IX. Besetzung der Landeschef und Ständepräsidentenposten in Prag; böhmische Deputation in Wien</label>
            </event>
          </listEvent>
          <listPerson type="attendants">
            <person role="chair"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8141"><surname>Kolowrat</surname></persName></person>
            <person role="attendant"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8145"><surname>Ficquelmont</surname></persName></person>
            <person role="attendant"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8154"><surname>Taaffe</surname></persName></person>
            <person role="attendant"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8148"><surname>Pillersdorf</surname></persName></person>
            <person role="attendant"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8151"><surname>Sommaruga</surname></persName></person>
            <person role="signed-protocol"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/8141"><surname>Kolowrat</surname></persName><note>BdE. <date when="1848-04-02">2. 4.</date></note></person>
            <person role="signed-protocol"><persName ref="https://mpr.acdh.oeaw.ac.at/entity/9251"><surname>Franz Karl</surname></persName><note>BdE. <date when="1848-04-02">2. 4.</date></note></person>
          </listPerson>
          <listPlace>
            <place xml:id="Wien">
              <placeName xml:lang="de">Wien</placeName>
              <placeName xml:lang="en">Vienna</placeName>
              <location>
                 <geo>48.208199 16.37169</geo>
              </location>
              <idno type="GeoNames">https://sws.geonames.org/2761369</idno>
           </place>
          </listPlace>
        </event>
        <!-- ... -->
      </listEvent>
    </egXML>
  </exemplum>

included a listPlace as suggested
expanded the references to the persons as we record them in the @oeaw-ministerratsprotokolle edition
removed the @ref/@when/@where attributes from the sub-events for brevity

I also considered removing a few agenda items, which can still be done, but they are funny enough to read and show the diversity of the concrete agenda of this body of government. Feel free to just delete a few if brevity is the goal…

skurzinz · 2023-09-05T14:35:47Z

@joeytakeda re my previous comment I also committed that file into my branch at skurzinz@85f0774

@skurzinz

* Updating example and German gloss (thanks, @skurzinz!)

ebeshero · 2023-11-07T15:04:12Z

@joeytakeda Is this now complete with the merging of the PR for eventName? (The last task unmarked here was updating all the <desc> elements, and I believe that's done... I'm provisionally putting a Guidelines milestone on this ticket...

ebeshero · 2023-11-07T15:04:24Z

@joeytakeda Just close if it's complete, okay?

joeytakeda added Priority: High Status: Go TEI: Guidelines & Documentation labels Sep 5, 2023

joeytakeda self-assigned this Sep 5, 2023

joeytakeda added a commit that referenced this issue Sep 5, 2023

Reordering classes of event and eventName (per #2466)

decb388

joeytakeda added a commit that referenced this issue Sep 5, 2023

Updating (English) desc for event per #2466

1134a14

joeytakeda added a commit that referenced this issue Oct 6, 2023

Continued work on event and eventName (#2382, #2466)

1fcf6f7

* Updating example and German gloss (thanks, @skurzinz!)

joeytakeda mentioned this issue Oct 6, 2023

Add eventName and update event #2483

Merged

ebeshero added this to the Guidelines 4.7.0 milestone Nov 7, 2023

joeytakeda closed this as completed Nov 7, 2023

ebeshero added this to the Guidelines 4.7.0 milestone Nov 17, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Make small fixes to event and eventName #2466

Make small fixes to event and eventName #2466

joeytakeda commented Sep 5, 2023 •

edited

Loading

skurzinz commented Sep 5, 2023 •

edited

Loading

joeytakeda commented Sep 5, 2023 •

edited

Loading

skurzinz commented Sep 5, 2023

skurzinz commented Sep 5, 2023

ebeshero commented Nov 7, 2023

ebeshero commented Nov 7, 2023

Make small fixes to event and eventName #2466

Make small fixes to event and eventName #2466

Comments

joeytakeda commented Sep 5, 2023 • edited Loading

skurzinz commented Sep 5, 2023 • edited Loading

joeytakeda commented Sep 5, 2023 • edited Loading

skurzinz commented Sep 5, 2023

skurzinz commented Sep 5, 2023

ebeshero commented Nov 7, 2023

ebeshero commented Nov 7, 2023

joeytakeda commented Sep 5, 2023 •

edited

Loading

skurzinz commented Sep 5, 2023 •

edited

Loading

joeytakeda commented Sep 5, 2023 •

edited

Loading