Open jacobwegner opened 4 years ago
@jabrah The current plugin parses the CTS URN to guess at the intended target annotation, which may not always be correct
I'll take another look at the URNs in the Georeference WA.
Part of the issue is that the subreferences from that WA are targeting a different translation ("Barber", perseus-eng4:) than the translation that is in the translation alignment WA ("Murray", perseus-eng3:).
The CITE URNs we're using to key off the translation alignments assume the Greek Venetus A transcription aligned to the Murray English translation, so it is possible that a place mentioned in the Murray translation may not exist verbatim in the barber (or not within the same text part designated by the URN).
As far as adding the new targets you're suggesting, I think for the named entities we're providing (e.g. https://aniop-atlas-staging.eldarion.com/wa/urn:cite2:hmt:msA.v1:12r/named-entities/collection/compound/0/) we could take one of two approaches
On the Georeference WA, we could fake "#1" above (because of the slight differences between the Barber and Murray translations) or we could create an annotation for just the Barber content within a given folio.
@jabrah One other note...I had been thinking about dropping the text
version of the translation alignment annotation (as discussed on #89).
If we're going to use the TextQuoteSelector
, would that work well enough for you with the HTML version of the translation alignment annotation?
I don't know about the content itself, so it's useful to have these sorts of insights,
TextQuoteSelector
would work fine with the HTML versionThere could be some interesting interactions bringing in a single annotation with the Greek content, then perhaps having the translation alignment annotations targeting that, along with named entities, etc. Probably out of scope though :)
I agree it'd be nice to have, but out of scope; we'd really need a new alignment for the Barber english translation to the Greek for that to work.
I'll work on updating our alignments as you've suggested this week:
@jabrah as an update, I wasn't able to get to the target and TextQuoteSelector changes this week, but plan on picking it back up next week.
(Unlike #89, there shouldn't be any backwards incompatible changes)
@jabrah I haven't made it to the changes discussed here. Would they still be helpful to you?
In future development, but not immediately, I would be comfortable deferring this until later
(Branched from #89)
@jabrah wrote: