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

update destatis #43

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

update destatis #43

wants to merge 16 commits into from

Conversation

maipet
Copy link

@maipet maipet commented Jul 8, 2024

Related to #41

@maipet maipet requested a review from acka47 July 8, 2024 11:43
@maipet
Copy link
Author

maipet commented Sep 11, 2024

@acka47 as discussed, the language labels for "en" and "uk" from the current hfs have been added and the duplicate notation "10" deleted

Copy link
Member

@acka47 acka47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das sieht super aus. Und da die Label jetzt je eine eigene Zeile haben, werden die zukünftigen Diffs noch sprechender sein, oder sehe ich da was falsch. So können wir das auf jeden Fall beim nächsten Treffen zum Mergen vorschlagen.

Allerdings gibt es noch ein Problem mit dem SkoHub Build, das ich nicht verstehe, siehe https://skohub.io/build/?id=6f2d0537-c96b-43c3-b7e9-fda8a8a8fb20

(Ich hatte den Build auch ein zweites Mal angestoßen, und es kam die gleiche Meldung.)

@acka47 acka47 assigned maipet and unassigned acka47 Sep 11, 2024
@acka47
Copy link
Member

acka47 commented Sep 11, 2024

Ah, das Problem ist glaube ich, dass die @baseURI fehlt.

@acka47
Copy link
Member

acka47 commented Sep 11, 2024

@acka47
Copy link
Member

acka47 commented Sep 30, 2024

Hier der Beginn einer Aufzählung, was der PR alles macht (du kannst den Kommentar gerne editieren, @maipet ):

  • neue Notationen ergänzen (welche?)
  • Das Turtle besser sortieren, damit sich in Zukunft diffs besser lesen lassen:
    • Labels für verschiedene Sprachen in je eine Zeile.
    • noch was?
  • narrower-Beziehungen entfernen, da redundant zu den inversen broader-Beziehungen, die drin bleiben. Damit wird die Pflege erleichtert.
  • Ergänzung zweier Skripte, die einige dieser Arbeiten übernehmen. Sollten wir die nicht auch in der README dokumentieren?

Beim Schreiben der Liste ist mir aufgefallen, dass die deprecated concepts aus der aktuellen Version in diesem Branch ganz verschwunden sind. Hier die entsprechenden Stellen im master Branch:

<n241> a skos:Concept ;
skos:prefLabel "Kerntechnik/Kernverfahrenstechnik"@de, "Nuclear technology/nuclear process engineering"@en, "Атомна інженерія/Інженерія ядерних процесів"@uk ;
skos:broader <n63> ;
skos:notation "241" ;
owl:deprecated true ;
skos:note "Integrated into 211"@en ;
skos:inScheme <scheme> .

<n237> a skos:Concept ;
skos:prefLabel "Mathematische Statistik/Wahrscheinlichkeitsrechnung"@de, "Mathematical statistics/probability calculation"@en, "Математична статистика/Теорія ймовірності"@uk ;
skos:broader <n37> ;
skos:notation "237" ;
owl:deprecated true ;
dct:isReplacedBy <n312> ;
skos:inScheme <scheme> .

Die müssen ja – als deprecated markiert – drinbleiben.

@maipet
Copy link
Author

maipet commented Oct 25, 2024

Ich hab nun doch noch die narrower relations ergänzt, da so ggf. gezielte Abfragen ermöglicht werden.
Ich bin nicht sicher, ob die Verlinkung auch zu den deprecated concept gesetzt werden soll? Gelten diese noch als narrower? Im Moment habe ich diese Beziehung nicht gesetzt, soll ich es ergänzen?

@acka47
Copy link
Member

acka47 commented Nov 7, 2024

Ich hab nun doch noch die narrower relations ergänzt, da so ggf. gezielte Abfragen ermöglicht werden.

Ok, wenn wir das ohnehin automatisch generieren, passieren ja auch keine Fehler und es macht keine Probleme bei der Pflege.

Ich bin nicht sicher, ob die Verlinkung auch zu den deprecated concept gesetzt werden soll? Gelten diese noch als narrower? Im Moment habe ich diese Beziehung nicht gesetzt, soll ich es ergänzen?

Es hat sich in der Vergangenheit gezeigt, dass die deprecated concepts weiterhin in der Hierarchie eingehänt sein sollten, siehe #35 (comment). Wir haben das dementsprechend auch in der StöberSpecs-Doku festgehalten (siehe dini-ag-kim/stoeberspecs#12), wo es jetzt heißt:

Veraltete SKOS-Konzepte werden als solche mit owl:deprecated true markiert und – so vorhanden – mit dct:isReplacedBy auf das Nachfolge-Konzept verlinkt. skos:broader- oder skos:narrower-Relationen der veralteten Konzepte sollen aber unangetastet bleiben.

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.

2 participants