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

Create architecture.md #139

Closed
wants to merge 3 commits into from
Closed

Create architecture.md #139

wants to merge 3 commits into from

Conversation

TobiasNx
Copy link
Contributor

@fsteeg I started off with a draft of this. Related to RPB-265

Could you have a look.

@TobiasNx TobiasNx marked this pull request as draft March 24, 2025 12:42
@TobiasNx
Copy link
Contributor Author

Besser?

flowchart BT
n1["Allegro"] -.Legacy Import.-> n4
 subgraph s0["Fremddaten"]
        n2["Lobid<br>Resources"]
        n3["Hebis"]
  end
 subgraph s1["Strapi"]
        n4["RPB"]
        n12["Manuelle Katalogisierung"]
        n11["Fremddatenübernahme"]
  end
n6["ElasticSearch<br>(lobid resources Basis)"]
 subgraph s3["Play"]
        n8["RPB"]
        n10["BiblioVino"]
  end

    n2 ----> n6
    n3 ----> n6
    n4 --> n6
    n6 --> n8
    n6 ---> n10
    n12 ---> n4
    n11 --RPB Klassifikation--> n4
    n11 -- Index Query--> n2
    n11 --SRU Query--> n3


    n2@{ shape: db}
    n3@{ shape: db}
    n6@{ shape: db}
    n8@{ shape: display}
    n10@{ shape: display}
    n1@{ shape: db}
    n11@{ shape: manual-input}
    n12@{ shape: manual-input}


Loading

@acka47
Copy link
Contributor

acka47 commented Mar 24, 2025

Ich finde die aktuelle Variante gut. Könnten wir meinetwegen erstmal so lassen.

@fsteeg
Copy link
Member

fsteeg commented Mar 24, 2025

Was mir auffällt:

  • Im Play-Kasten sollte lobid-resources-rpb ergänzt werden, darauf greifen RPB und BiblioVino zu, nur lobid-resources-rpb greift auf den Index zu
  • Im Index statt Basis lieber z.B. Index-Konfig (um deutlich zu machen, dass es nicht der lobid-resources-Index ist)
  • Die Pfeile von Fremddatenübernahme zu lobid-resources und Hebis scheinen mir in die falsche Richtung zu gehen
  • Die RPB-Klassifikation hat nichts mit der Fremddatenübernahme zu tun, "Fremddaten" ist ja eine eigene Maske
  • Stattdessen könnte man im Strapi-Kasten RPB-Normdaten (eine Maske, sichtbar) ergänzen
  • Die Klassifikationen sind auch eigene Typen in Strapi, aber nicht als Maske sichtbar (rpb-notationen, rpb-spatial), und werden aus den SKOS-Dateien auf GitHub eingespielt
  • Eventuell macht es Sinn, eher von den Masken auszugehen: Aufsätze, Selbständige Werke, Fremddaten statt RPB
  • Die RPPD aus dem ersten Diagramm muss ja nicht wegfallen, sondern sollte ja als eigene Skizze bleiben
  • Das verteile Review mit Feedback im Issue und Antwort hier im PR, und Verison 1 als Datei im Branch und Version 2 als Kommentar ist verwirrend, ich würde vorschlagen du machst beides in einer md-Datei, und passt die immer an

@TobiasNx
Copy link
Contributor Author

TobiasNx commented Mar 24, 2025

So ich hab es nochmal überarbeitet. Aktuell fehlt noch die RPPD. https://github.com/hbz/rpb/blob/architecture/architecture.md

@TobiasNx TobiasNx assigned fsteeg and acka47 and unassigned TobiasNx Mar 24, 2025
@fsteeg
Copy link
Member

fsteeg commented Mar 24, 2025

  • Das Dreieck "Datenübernahme mit Metafacture" kann weg, das ist kein eigenes Modul oder so, der Pfeil "Initalisierung einer Indexroutine" sollte in lobid-resources-rpb gehen, und von dort ein zusätzlicher Pfeil in den Index
  • Wie oben beschrieben sollte meiner Meinung nach RPB-Normdaten sowie rpb-notationen & rpb-spatial samt SKOS-Quelle ergänzt werden
  • Ich glaube, es erhöht die Übersicht, die veschiedenen Masken / Typen anzugeben, statt RPB, manuelle Katalogisierung und Fremddatenübernahme. Die verschiedenen Typen kommen ja aus verschiedenen Quellen und werden unterschiedlich genutzt, z.B. für Lookups. Dann sieht man eher die Komponenten des Systems.

@fsteeg fsteeg assigned TobiasNx and unassigned fsteeg Mar 25, 2025
@acka47
Copy link
Contributor

acka47 commented Mar 28, 2025

Closing in favour of Miro-based architecture outline.

@acka47 acka47 closed this Mar 28, 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

Successfully merging this pull request may close these issues.

3 participants