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

(phi0690) Vergil? #590

Open
lcerrato opened this issue Jan 22, 2025 · 2 comments
Open

(phi0690) Vergil? #590

lcerrato opened this issue Jan 22, 2025 · 2 comments
Assignees

Comments

@lcerrato
Copy link
Contributor

I really dislike the SV showing Virgil as the default. Not sure if this is from the cts metadata or the catalog, but it should either be the Roman name or Vergil. I think it's standard that Virgil is a later invention?

@AlisonBabeu
Copy link
Contributor

AlisonBabeu commented Jan 23, 2025

hi @lcerrato well it looks like the only Vergil/Virgil/P. Vergilius Maro cts_xml file is in Perseus Canonical Latin (since all the Scaife versions are the updated Perseus files)

The file here:https://github.com/PerseusDL/canonical-latinLit/blob/master/data/phi0690/__cts__.xml has three names:

<ti:groupname xml:lang="lat">P. Vergilius Maro</ti:groupname>
<ti:groupname xml:lang="eng">Vergil</ti:groupname>
<ti:groupname xml:lang="eng">Virgil</ti:groupname>

I'm not sure why Scaife is using Virgil, perhaps because it is the last name in the file? I'm going to update this to only include Vergil if you prefer and that should fix the issue.

@lcerrato
Copy link
Contributor Author

@AlisonBabeu Yes, I was wondering about that. I can't complete the build so I cannot even test this at the moment which is why I thought I'd consult you first. I don't know why it would take the last one.

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

No branches or pull requests

2 participants