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

Normalize <said><label/>...</said> interaction #1550

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

npmccallum
Copy link
Contributor

Documents in this repository tend to follow a format that looks like this:

<said who="#NAME">
  <label>NAME_OR_ABBR.</label>
  ...
</said>
  1. The who attribute starts with # and contains the full name.
  2. The label element contains the name or an abbr., followed by a period.

This commit fixes the handful of instances that deviate from this pattern.

Documents in this repository tend to follow a format that looks like this:

```
<said who="#NAME">
  <label>NAME_OR_ABBR.</label>
  ...
</said>
```

1. The `who` attribute starts with `#` and contains the full name.
2. The `label` element contains the name or an abbr., followed by a period.

This commit fixes the handful of instances that deviate from this pattern.
@@ -96,7 +96,7 @@ schematypens="http://relaxng.org/ns/structure/1.0"?>
<div type="textpart" subtype="section" n="intro"> <head>INTRODUCTION</head>
<p rend="indent">Plutarch’s essay on the changed custom at Delphi is quite as interesting for its digressions as for its treatment of the main topic. Portents, coincidences, history, a little philosophy, stories of persons like Croesus, Battus, Lysander, Rhodope, finally lead up to the statement that many oracles used to be delivered in prose, although still more in early times were delivered in verse; but the present age calls for simplicity and directness instead of the ancient obscurity and grandiloquence. </p> <p rend="indent"> We possess a considerable body of Delphic oracles preserved in Greek literature, as, for example, the famous oracle of the <q>wooden wall</q> (Herodotus, vii. 141). Practically all of these are in hexameter verse. Many more records of oracles merely state that someone consulted the oracle and was told to perform a certain deed, or was told that something would or might happen, often with certain limitations. We have, therefore, no means of determining the truth of Plutarch’s statement, but there is little doubt that he is right. If we possessed his lost work, <foreign xml:lang="grc">Χρησμῶν συναγωγή</foreign> (no. 171 in Lamprias’s list), we should have more abundant data on which to base our decision. </p> <p rend="indent"> The essay often exhibits Plutarch at his best. Hartman thinks that Plutarch hoped that the.work <pb xml:id="v.5.p.257"/> would be read at Rome, and therefore inserted the encomium of Roman rule near the end. </p> <p rend="indent"> The essay stands as no. 116 in Lamprias’s catalogue. It is found in only two mss. and in a few places the tradition leaves us in doubt, but, for the most part, the text is fairly clear. </p> <p rend="indent"> The references to the topography and monuments of Delphi have become more intelligible since the site was excavated by the French. Pomtow, in the <title xml:lang="deu" rend="italic">Berliner Pkilologische Wochenschrift</title>, 1912, p. 1170, gives an account of the monuments visited by the company in this essay. </p> </div> <pb xml:id="v.5.p.259"/>
<div type="textpart" subtype="section" n="1"><p rend="center"><label>(The persons who take part in the dialogue are Basilocles and Philinus, who serve to introduce the later speakers; Diogenianus, Theon, Sarapion, Boethus, as well as Philinus himself and some professional guides.)</label></p>
<p rend="indent"><said who="#Basilocles"><label>BASILOCLES.</label> You people have kept it up till well into the evening, Philinus, escorting the foreign visitor around among the statues and votive offerings. For my part, I had almost given up waiting for you.</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> The fact is, Basilocles, that wre went slowly, sowing words, and reaping them straightway with strife, like the men sprung from the Dragon’s teeth, words with meanings behind them of the contentious sort, which sprang up and flourished along our way.</said> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES.</label> Will it be necessary to call in someone else of those who were with you; or are you willing, as a favour, to relate in full what your conversation was and who took part in it?</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> It looks, Basilocles, as if I shall have that to do. In fact, it would not be easy for you to find anyone of the others in the town, for I saw most of them once more on their way up to the Cory ei an cave and Lycoreia<note resp="editor" place="unspecified" anchored="true">Pausanias, x. 6. 2-3.</note> with the foreign visitor.</said> <pb xml:id="v.5.p.261"/> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES</label>. Our visitor is certainly eager to see the sights, and an unusually eager listener.</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> But even more is he a scholar and a student. However, it is not this that most deserves our admiration, but a winning gentleness, and his willingness to argue and to raise questions, which comes from his intelligence, and shows no dissatisfaction nor contrariety with the answers. So, after being with him but a short time, one would say, <q>O child of a goodly father!</q> <note resp="editor" place="unspecified" anchored="true"> <foreign xml:lang="lat">Cf.</foreign> Plato, <title rend="italic">Republic</title>, 368 a.</note> You surely know Diogenianus, one of the best of men.</said> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES.</label> I never saw him myself, Philinus, but I have met many persons who expressed a strong approval of the man’s words and character, and who had other compliments of the same nature to say of the young man. But, my friend, what was the beginning and occasion of your conversation?</said> </p> </div>
<p rend="indent"><said who="#Basilocles"><label>BASILOCLES.</label> You people have kept it up till well into the evening, Philinus, escorting the foreign visitor around among the statues and votive offerings. For my part, I had almost given up waiting for you.</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> The fact is, Basilocles, that wre went slowly, sowing words, and reaping them straightway with strife, like the men sprung from the Dragon’s teeth, words with meanings behind them of the contentious sort, which sprang up and flourished along our way.</said> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES.</label> Will it be necessary to call in someone else of those who were with you; or are you willing, as a favour, to relate in full what your conversation was and who took part in it?</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> It looks, Basilocles, as if I shall have that to do. In fact, it would not be easy for you to find anyone of the others in the town, for I saw most of them once more on their way up to the Cory ei an cave and Lycoreia<note resp="editor" place="unspecified" anchored="true">Pausanias, x. 6. 2-3.</note> with the foreign visitor.</said> <pb xml:id="v.5.p.261"/> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES.</label> Our visitor is certainly eager to see the sights, and an unusually eager listener.</said> </p> <p rend="indent"> <said who="#Philinus"><label>PHILINUS.</label> But even more is he a scholar and a student. However, it is not this that most deserves our admiration, but a winning gentleness, and his willingness to argue and to raise questions, which comes from his intelligence, and shows no dissatisfaction nor contrariety with the answers. So, after being with him but a short time, one would say, <q>O child of a goodly father!</q> <note resp="editor" place="unspecified" anchored="true"> <foreign xml:lang="lat">Cf.</foreign> Plato, <title rend="italic">Republic</title>, 368 a.</note> You surely know Diogenianus, one of the best of men.</said> </p> <p rend="indent"> <said who="#Basilocles"><label>BASILOCLES.</label> I never saw him myself, Philinus, but I have met many persons who expressed a strong approval of the man’s words and character, and who had other compliments of the same nature to say of the young man. But, my friend, what was the beginning and occasion of your conversation?</said> </p> </div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am having trouble finding any difference here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Friendly tip: using git diff --word-diff-regex=. makes these kind of changes much easier to see. However, the change in the above line is this:

- <label>BASILOCLES</label>.
+ <label>BASILOCLES.</label>

@lcerrato
Copy link
Collaborator

@npmccallum
I am not sure about the changes here? Some of the names are fully spelled out and others are abbreviated even within the first few files.

@npmccallum
Copy link
Contributor Author

@npmccallum I am not sure about the changes here? Some of the names are fully spelled out and others are abbreviated even within the first few files.

This is a pre-existing stylistic choice inconsistency which I am not attempting to address in this PR. This PR attempts only to address syntactical inconsistencies.

@lcerrato
Copy link
Collaborator

@npmccallum
Yes, but I don't see any changes in some of these diffs? Notably the first diff in the first file: I don't see a change there?
I also see both expanded labels and condensed in the same file(s)?
Earlier conversion work, such as Plato will be reviewed as part of this workflow.

@npmccallum
Copy link
Contributor Author

npmccallum commented Nov 24, 2023

I also see both expanded labels and condensed in the same file(s)? Earlier conversion work, such as Plato will be reviewed as part of this workflow.

Yes, the existing files are inconsistent. I'm not attempting to address this particular inconsistency in this PR. I could file an issue for this.

I don't see any changes in some of these diffs? Notably the first diff in the first file: I don't see a change there?

This appears to be a bug with GitHub's HTML diff rendering. I see the changes easily using git diff --word-diff-regex=.. Here's a manual diff created by hand of all the changes:

  1. data/tlg0007/tlg091/tlg0007.tlg091.perseus-eng3.xml - Move the period inside the label tag.
- <label>BASILOCLES</label>.
+ <label>BASILOCLES.</label>
  1. data/tlg0007/tlg091/tlg0007.tlg091.perseus-grc2.xml - Add a missing period.
- <label>ΦΙΛΙΝΟΣ</label>
+ <label>ΦΙΛΙΝΟΣ.</label>
  1. data/tlg0007/tlg138/tlg0007.tlg138.perseus-eng2.xml - Remove a stray period.
- <label>LAMPRIAS.</label> . 
+ <label>LAMPRIAS.</label>
  1. data/tlg0007/tlg138/tlg0007.tlg138.perseus-eng2.xml - Add a missing name. Move stray period.
- <label></label> . 
+ <label>LAMPRIAS.</label>
  1. data/tlg0059/tlg005/tlg0059.tlg005.perseus-eng2.xml - Add a missing hash.
- <said who="Socrates"
+ <said who="#Socrates"
  1. data/tlg0059/tlg005/tlg0059.tlg005.perseus-eng2.xml - Add missing period. (twice)
- <label>Socrates</label>
+ <label>Socrates.</label>
  1. data/tlg0059/tlg019/tlg0059.tlg019.perseus-eng2.xml - Add missing period.
- <label>Soc</label>
+ <label>Soc.</label>
  1. data/tlg0059/tlg032/tlg0059.tlg032.perseus-eng2.xml - Remove stray (extra) period.
- <label>Crit..</label>
+ <label>Crit.</label>

@npmccallum
Copy link
Contributor Author

@lcerrato Any progress on this?

@lcerrato
Copy link
Collaborator

@npmccallum
I would prefer to look at this in the context of making all necessary changes at once rather than a piecemeal effort, particularly for published "live" texts.
This PR spans a few different author groups, so I want to see these in those contexts rather than jumping in and out of various corpora with no sense of things.
(The unpublished translations weren't going to matter in the short term.)

(Also am on a limited schedule + holidays + other projects/deadlines).

@lcerrato lcerrato merged commit 8930a8d into PerseusDL:master Dec 4, 2023
2 checks passed
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.

2 participants