hbz / nwbib

Die Nordrhein-Westfälische Bibliographie
http://nwbib.de
3 stars 2 forks source link

Knittkuhl (Q44023866) wurde mit Q17351610 gemergt #598

Closed acka47 closed 1 year ago

acka47 commented 1 year ago

Wir wurden erfolgreich durch unser Monitoring ("Alert nwbib-spatial") darauf aufmerksam gemacht, erstmals am 2023-01-22.

Hier die Änderung: https://www.wikidata.org/w/index.php?title=Q17351610&oldid=1815944157

Wir müssen das analog zu #593 anpassen.

acka47 commented 1 year ago

@fsteeg schrieb in https://github.com/hbz/nwbib/issues/593#issuecomment-1323990833:

Da wir beim nwbib-spatial-conf-Ansatz dann ja wieder schauen müssten, wie wir entscheiden, dass wir den 'echten' Wikidata-Eintrag nicht wollen, habe ich jetzt mal deinen Ansatz aus https://github.com/hbz/nwbib/issues/593#issuecomment-1303115080 umgesetzt, d.h. wir nehmen generell die Wikidata-nwbibId als ID für unsere concepts, und die Wikidata-item-ID als focus: https://github.com/hbz/nwbib/commit/cd73342bd975e2d4be4a6706c310496f3081d9db.

Wenn ich das richtig verstehe, dürfte – vorausgesetzt wir hätten das wie dort beschrieben umgesetzt – eigentlich gar keine Fehlermeldung kommen, weil uns die SPARQL Query( code, Wikidata Query Service) eigentlich die richtigen Daten zum korrekten Bau der SKOS-Datei liefert:

image

Irgendetwas scheint da noch nicht ganz rund zu laufen, oder @fsteeg ?

acka47 commented 1 year ago

Irgendetwas scheint da noch nicht ganz rund zu laufen, oder @fsteeg ?

Ich habe falsch gedacht. Das Alert-Skript hat uns korrekterweise auf den Merge hingewiesen. Wir müssen das nun bei uns umsetzen. Deshalb habe ich gerade einmal den Aktualisierungsprozess laufen lassen und die Änderungen commited: https://github.com/hbz/lobid-vocabs/commit/eb23a4e6e0570f0417975687ac01a9fea3aa2fdc

Reicht das jetzt bereits aus, damit die Alert-Mail nicht mehr getriggert wird? Wir werden sehen.

fsteeg commented 1 year ago

Reicht das jetzt bereits aus, damit die Alert-Mail nicht mehr getriggert wird? Wir werden sehen.

Sieht so aus, und ergibt auch Sinn, das Skript baut die Klassifikation selbst (sbt "runMain SpatialToSkos"): https://github.com/hbz/nwbib/blob/master/cron.sh

acka47 commented 1 year ago

Sieht so aus, und ergibt auch Sinn, das Skript baut die Klassifikation selbst (sbt "runMain SpatialToSkos"): https://github.com/hbz/nwbib/blob/master/cron.sh

Hmm, dann wundert es mich aber, warum überhaupt eine Alert-Mail rausgeschickt wurde. irgendwie muss doch auch mit der aktuellen nwbib-spatial.ttl verglichen werden, oder wie wurde die Mail dann getriggert?

fsteeg commented 1 year ago

Das Alert kam doch bevor du die Klassifikation gestern neu gebaut und deployed hattest oder?

acka47 commented 1 year ago

Das Alert kam doch bevor du die Klassifikation gestern neu gebaut und deployed hattest oder?

Ja, ich hatte allerdings nur neu gebaut, nicht deployt. (Die SKOS-Datei auf der Webseite ist allerdings aktuell. 🤔 ) Wenn das Vergleichsskript aber selbst baut, sollte es ja irrelevant, was im lobid-vocabs-Repo liegt.

fsteeg commented 1 year ago

Wenn das Vergleichsskript aber selbst baut, sollte es ja irrelevant, was im lobid-vocabs-Repo liegt.

Aso, ne, der triggert das Bauen, aber die Production-Instanz nutzt ja den lobid-vocabs-master, d.h. der baut dann den statischen Stand von https://raw.githubusercontent.com/hbz/lobid-vocabs/master/nwbib/nwbib-spatial.ttl (beim Bauen erzeugt er die Dateien, die dann ausgewertet werden um das Alert zu schicken).

acka47 commented 1 year ago

Zusammenfassend, um sicherzugehen, dass ich es richtig verstanden habe: Wenn das nächste Mal eine Altert-Mail kommt wegen eines Merges von Wikidata-Items, dann reicht es erstmal aus https://github.com/hbz/lobid-vocabs/blob/master/nwbib/nwbib-spatial.ttl zu aktualisieren, damit zumindest die E-Mail nicht mehr kommt. Das Deployment können wir dann ja auch zusammen mit den nächsten größeren Änderungen machen. Ja?

fsteeg commented 1 year ago

Ja, klingt gut, wenn du damit meinst dass du https://github.com/hbz/lobid-vocabs/blob/master/nwbib/nwbib-spatial.ttl per sbt "runMain SpatialToSkos" aktualisiert hast (nicht manuell editiert).

acka47 commented 1 year ago

Ja, klingt gut, wenn du damit meinst dass du https://github.com/hbz/lobid-vocabs/blob/master/nwbib/nwbib-spatial.ttl per sbt "runMain SpatialToSkos" aktualisiert hast (nicht manuell editiert).

Ich habe die Änderung per runMain SpatialToSkos generiert und per git push im Repo aktualisiert.

acka47 commented 1 year ago

Closing