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

Possible logic error in the indexer #191

Open
jcowey opened this issue Feb 18, 2025 · 0 comments
Open

Possible logic error in the indexer #191

jcowey opened this issue Feb 18, 2025 · 0 comments
Assignees

Comments

@jcowey
Copy link
Member

jcowey commented Feb 18, 2025

Starting point is: https://papyri.info/ddbdp/p.tor.choach;;8a

Files involved are:

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/p.tor.choach/p.tor.choach.8a.xml

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/p.tor.choach/p.tor.choach.8b.xml

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/upz/upz.2/upz.2.170.xml

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/p.paris/p.paris.14.xml

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/p.tor/p.tor.3.xml

p.tor.3 => upz.2.170 A

p.paris.14 => upz.2.170 B

upz.2.170 A => p.tor.choach.8a (TM 3571)

upz.2.170 B => p.tor.choach.8b (TM 3638)

From the legacy data in sgml

<div1 type="document" n="170">
<head lang=en>
<date>120BC</date>
<placeName>Thebes</placeName>
<xref type="reprint in">P.Tor.Choach 8A</xref>
</head>
<p lang=greek>
<milestone unit="4" n="A">
<lb n="1"><foreign lang="la">UPZ <num>2</num>,<num>170</num>A%6PTorChoach <num>8</num>A </foreign>
<milestone unit="4" n="Brp">
<lb n="1"><note>UPZ 2,170B = <xref>PTorChoach 8B</xref></note>
</div1>

we can see that upz.2.170 was only ever a redirect.

At some stage it received the TM number 3638

https://github.com/papyri/idp.data/blob/master/DDB_EpiDoc_XML/upz/upz.2/upz.2.170.xml#L18

This is a case where one file (with one TM number, it should have two TM numbers) was reprinted in two separate files each with its own correct TM number.

How to deal with this?

@jcowey jcowey assigned jcowey and hcayless and unassigned jcowey Feb 18, 2025
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