OpenGreekAndLatin / First1KGreek

XML files for the works in the First Thousand Years of Greek Project. Please see our Wiki on how to contribute.
https://opengreekandlatin.github.io/First1KGreek/
Creative Commons Attribution Share Alike 4.0 International
91 stars 85 forks source link

(tlg4034)_all_files #2714

Closed AlisonBabeu closed 1 year ago

AlisonBabeu commented 1 year ago

Updated URNs and cts_metadata.

lcerrato commented 1 year ago

@AlisonBabeu There are two errors here, looks like misplaced tags (missing "imprint"?)

tlg4034.tlg004.1st1K-grc1.xml error: element "publisher" not allowed here [In (L61 C44)], error: element "pubPlace" not allowed here [In (L62 C43)], error: element "date" not allowed here [In (L63 C39)], error: element "monogr" incomplete [In (L65 C34)] 2460 tlg4034.tlg005.1st1K-grc1.xml error: element "publisher" not allowed here [In (L61 C44)], error: element "pubPlace" not allowed here [In (L62 C43)], error: element "date" not allowed here [In (L63 C38)], error: element "monogr" incomplete [In (L64 C34)]

AlisonBabeu commented 1 year ago

hi @lcerrato. How strange, you know when I was editing the files, I got a STOA message that the schema couldn't be found. I must have messed up the tags and it didn't catch them because of that. Sorry I will go fix that!

lcerrato commented 1 year ago

@AlisonBabeu Yes, the EpiDoc site has been on and offline but it is built in to the Hooktest, so that will show the errors. I don't know why it shows a "successful" test above, but I think we should ask Jake on that. It used to be that anything less than 100% was a failure.

AlisonBabeu commented 1 year ago

hi @lcerrato should I open an issue in Scaife viewer for that? I won't be at the developers meeting next week!

lcerrato commented 1 year ago

@AlisonBabeu No, I can see the parameter for this. I may try an experiment when you are away before that. I also think it won't do the release if it fails, so the errors may show up on the merge attempt.

Just check the hook test results as you go.

lcerrato commented 1 year ago

@AlisonBabeu Long term the stoa site is being unreliable so we may want to host the schema locally or change to https://epidoc.stoa.org/schema/latest/tei-epidoc.rng

As there are thousands of files this would be a massive change, so that's why I did not adopt it.

This really is impeding my work, though, as everything is hanging on this link.

lcerrato commented 1 year ago

The errors to the tlg4016 files were in the first pull request, too. I just spotted the errors to the tlg4034 first.

This is a problem with this testing regime: previously, if things were not 100% we would see an error. I guess that does not happen now, so the error here was probably introduced some time ago and flew under the radar because the tests read "passed."

lcerrato commented 1 year ago

In tlg4016.tlg002.1st1K-grc1.xml <div type="edition" n="urn:cts:greekLit:tlg4016.tlg002.1st1K-grc2" xml:lang="grc"> where "grc2" is incorrect tlg4016.tlg004.1st1K-grc1.xml metadata file was not updated to reflect file name change

AlisonBabeu commented 1 year ago

hey @lcerrato sorry for all this. I'm going to hold off on any more changes like this until I'm back and either have a local copy of the schema to test against or if STOA proves more reliable! I will, however, fix these!

AlisonBabeu commented 1 year ago

hey @lcerrato so I believe my last pull request here fixed issues on tlg4034 and I opened a separate pull request to fix the ones in tlg4016.

lcerrato commented 1 year ago

@AlisonBabeu I will keep an eye on this.