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.

Sign up for GitHub

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

Jump to bottom

Marriages setup #23

Closed
vanlummelhuizen opened this issue Jul 8, 2024 · 23 comments
Closed

Marriages setup #23

vanlummelhuizen opened this issue Jul 8, 2024 · 23 comments
Assignees
@ACNMontoya @dennis-bor

Comments

@vanlummelhuizen
Copy link
Member

@ACNMontoya @dennis-bor

Just now I added a Marriage tab to the admin page for changing a Person on the test server (e.g. https://test.shewrote.rich.ru.nl/admin/shewrote/person/e98cf5f8-9a9b-4c5e-992c-b4d36a7eb50c/change/#marriages).

When creating a Marriage (click on '+ Add another Marriage') there are actually two Marriage objects created, one for each direction. Updating and deleting one direction should also update or delete the other direction.

Note that this setup does not account for the occasion that a couple are married multiple times. See also our mail correspondence on april 11, 2024 with subject Shewrote: meerdere huwelijken tussen twee mensen?.

Let me know what you think.
@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Jul 8, 2024 via email

@vanlummelhuizen
Copy link
Member Author

Ja, daar ga ik naar kijken.

@vanlummelhuizen
Copy link
Member Author

Ik kan in de data van Womenwriters geen zaken vinden die een marriage (= Person-Person-relatie) aangeven. Alleen 'Marital status' en Spouse bij Temporary data (zie bv https://womenwriters.rich.ru.nl/womenwriters/vre/persons/0cba6668-ed04-4df6-8021-7e389ad12b9a/personal-situation). Met alleen een spouse-naam, dus geen ID, kan ik geen script maken om marriage-data te importeren. Kijk ik ergens overheen?

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Jul 8, 2024 via email

@vanlummelhuizen
Copy link
Member Author

Ik zie wel dat 'spouse' een aparte veld is in de 1e versie van de database, zie bijv. http://neww.huygens.knaw.nl/authors/show/7, help dat?

Het lijkt erop dat dat ook alleen een naam is en geen ID ofzo. We moeten misschien ook even de dump van het Huygens afwachten.

Ik kan overigens altijd met de tekst in Spouse van de Temporary data een nieuwe Person aanmaken. Die is dat verder wel leeg en een eventuele Marriage met een bestaande Person object wordt dan niet gemaakt.

@vanlummelhuizen
Copy link
Member Author

Ik heb hier opnieuw naar gekeken en daarbij de datadump van de eerste versie van Womenwriters meegenomen. Ook die laatste heeft slechts een tekst-veld met de naam van de spouse.

Een kleine steekproef laat zien dat de spouse-namen niet naar een bestaande author/person-object verwijzen. Met andere woorden, een Person-Person-relatie zit er sowieso niet in.

Ik kan natuurlijk de spouse-naam gebruiken om daar een nieuwe -verder lege- Person-object van te maken. Het is dat aan editors om de info van die Person-objecten verder aan te vullen, indien nodig.

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 20, 2024 via email •
edited by vanlummelhuizen
Loading

@vanlummelhuizen
Copy link
Member Author

De Person-Person-relatie kan ik wel maken, maar die bevat dan niet meer dan een short_name, gewoonweg omdat er niet meer in WomenWriters zit dan dat.

Ik heb even gekeken of ik Geertruida Bosboom-Toussaint kon vinden en volgens mij is dit haar: https://test.shewrote.rich.ru.nl/persons/dbab0086-38dc-4ee0-8d21-b9826e2f3d19/.

De reden dat ze niet met 'Bosboom' te vinden is, is dat in de laatste WomenWriters twee name componenten zitten en alleen de eerste gebruikt is bij het importeren:

"names": [
      {
        "components": [
          {
            "type": "FORENAME",
            "value": "Anna"
          },
          {
            "type": "FORENAME",
            "value": "Louisa"
          },
          {
            "type": "FORENAME",
            "value": "Geertruida"
          },
          {
            "type": "SURNAME",
            "value": "Toussaint"
          }
        ]
      },
      {
        "components": [
          {
            "type": "FORENAME",
            "value": "Anna"
          },
          {
            "type": "FORENAME",
            "value": "Louisa"
          },
          {
            "type": "FORENAME",
            "value": "Geertruida"
          },
          {
            "type": "SURNAME",
            "value": "Bosboom"
          },
          {
            "type": "SURNAME",
            "value": "- Toussaint"
          }
        ]
      }
    ],

Overigens heb ik bij alle objecten uit de laatste WomenWriters die geïmporteerd zijn, ook de originele data opgenomen. We kunnen dus daar makkelijk in kijken om er meer data uit te halen.

Overigens kan ik IDs die je noemt nergens vinden. Waar verwijzen ze precies naar?

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 20, 2024 via email

vanlummelhuizen added a commit that referenced this issue Aug 21, 2024
@vanlummelhuizen
feat: Import alternative names and add to search #23
e8c90e6
@vanlummelhuizen
Copy link
Member Author

Ik heb AlternativeName-objecten aangemaakt mbv de originele data. Ook wordt er nu gezocht in AlternativeNames als je zoek op naam. Zie bv https://test.shewrote.rich.ru.nl/persons/?short_name=bosboom

Het is nu alleen nog op de test-server. Ik hoor graag of het ook naar de live-server kan.

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 21, 2024 via email

@vanlummelhuizen
Copy link
Member Author

Het kan op de live-server.

Gedaan.

Overigens, het is een andere Issue, maar 'maiden name' moet heten 'birth name'.

Dat staat in de planning: #26

vanlummelhuizen added a commit that referenced this issue Aug 21, 2024
@vanlummelhuizen
feat: Import spouse info and create marriages #23
2f18015
@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 21, 2024 via email

@vanlummelhuizen
Copy link
Member Author

Ik heb inmiddels ook Marriage-objecten aangemaakt met de tempSpouse data. Zie bv https://test.shewrote.rich.ru.nl/persons/be085122-844d-4103-9377-2102abe8914a/#v-pills-personal: Margaret Maria Brewster is getrouwd met ene Gordon. Van die laatste heb ik dus een verder leeg Person-object gemaakt. Zie ook https://womenwriters.rich.ru.nl/womenwriters/vre/persons/be085122-844d-4103-9377-2102abe8914a/personal-situation.

Ik hoor weer graag of dit in orde is.

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 21, 2024 via email

@vanlummelhuizen
Copy link
Member Author

Dank hiervoor, dit lijkt mij goed zo.

Dit heb ik zojuist ook op de live-server gedaan.

@vanlummelhuizen
Copy link
Member Author

Ik zie dat er heel veel lege Person-objecten zijn aangemaakt, omdat ieder Person-object in WomenWriters een tempSpouse-veld had ook al was die leeg. Ik kan deze Person-objecten en de Marriage-objecten waar ze in zitten verwijderen. Goed idee?

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 22, 2024 via email

vanlummelhuizen added a commit that referenced this issue Aug 22, 2024
@vanlummelhuizen
fix: Discard empty tempSpouse entries #23
54f9156
@vanlummelhuizen
Copy link
Member Author

Ik test dat short_name, first_name en birth_name (voorheen maiden_name) leeg zijn. Zouden er Person-objecten kunnen zijn waarbij deze 3 velden leeg zijn maar toch moeten blijven bestaan?

@vanlummelhuizen
Copy link
Member Author

Ik heb de test voor lege Person-objecten uitgebreid met bijna alle velden. Ik heb dit inmiddels op de test-server uitgevoerd. Kan je kijken of je dingen mist nu?

@ACNMontoya
Copy link
Collaborator

ACNMontoya commented Aug 22, 2024 via email

This was referenced Aug 22, 2024
Closed
Closed
Open
@vanlummelhuizen
Copy link
Member Author

Bij Marriages lijkt het te kloppen

Dan ga ik ook op de live-server de lege Person-objecten en bijbehorende Marriages verwijderen.

Van de rest van je opmerkingen heb ik nieuwe issues aangemaakt: #29, #30 en #31

@vanlummelhuizen
Copy link
Member Author

Dan ga ik ook op de live-server de lege Person-objecten en bijbehorende Marriages verwijderen.

Gedaan. Hierbij sluit ik dit issue.

@ACNMontoya ACNMontoya mentioned this issue Sep 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ACNMontoya ACNMontoya

@dennis-bor dennis-bor

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vanlummelhuizen @ACNMontoya @dennis-bor