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 19 commits into
base: master
Choose a base branch
from
Open

update destatis #43

wants to merge 19 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 ):

  • 2 Notationen korrigieren:
    • <n0128> a skos:Concept -> <n128> a skos:Concept
    • <n030010001> a skos:Concept -> <n292> a skos:Concept
  • Das Turtle besser sortieren, damit sich in Zukunft diffs besser lesen lassen:
    • Labels für verschiedene Sprachen in je eine Zeile.
    • broader- / narrower-Beziehungen in je einer Zeile
  • 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?
  • deprecated aus der vorherigen Version beibehalten

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.

@acka47 acka47 self-requested a review November 12, 2024 09:53
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.

+1

@acka47 acka47 requested a review from mic-men November 12, 2024 10:28
@acka47 acka47 assigned mic-men and unassigned maipet Nov 12, 2024
@oellers
Copy link
Member

oellers commented Nov 12, 2024

I'd suggest to eliminate the build warnings, e.g. by adding dct:license and dct:source.

Suggestion:

  • dct:license
    Find/Use an appropriate license URI, e.g. from DCT AP?
    http://dcat-ap.de/def/licenses/other-closed

  • dct:source
    The URI to the source
    https://www.destatis.de/DE/Methoden/Klassifikationen/Bildung/studenten-pruefungsstatistik.html

@acka47 acka47 removed the request for review from mic-men November 12, 2024 10:41
@acka47 acka47 assigned maipet and unassigned mic-men Nov 12, 2024
@acka47 acka47 self-requested a review November 12, 2024 10:41
@mic-men
Copy link
Contributor

mic-men commented Nov 20, 2024

Ich habe alles einmal durchgesehen hinsichtlich Logik und Tippfehler. Sieht für mich alles gut aus.
Allerdings hadere ich noch mit den Korrekturen bzgl. IDs n0128 und n030010001. Sachlich richtig, aber unsere eingesetzte Software berücksichtigt deprecated und replacedBy (bisher) nicht und es wird zu Problemen kommen. Ich möchte daher nochmal zur Diskussion stellen, ob wir die fehlerhaften IDs nicht doch (noch) beibehalten sollen?
Es ist ein Unterschied in der Praxis, ob ein eher nebensächliches Fach deprecated ist und irgendwo aufgeht oder ob ein essentielles Fach wie Physik eine andere ID bekommt.

@acka47
Copy link
Member

acka47 commented Dec 3, 2024

Allerdings hadere ich noch mit den Korrekturen bzgl. IDs n0128 und n030010001. Sachlich richtig, aber unsere eingesetzte Software berücksichtigt deprecated und replacedBy (bisher) nicht und es wird zu Problemen kommen. Ich möchte daher nochmal zur Diskussion stellen, ob wir die fehlerhaften IDs nicht doch (noch) beibehalten sollen?
Es ist ein Unterschied in der Praxis, ob ein eher nebensächliches Fach deprecated ist und irgendwo aufgeht oder ob ein essentielles Fach wie Physik eine andere ID bekommt.

Ich sehe das genauso. Vorschlag: Wir passen zumindest einmal n030010001 an zu n292, weil das ja komplett daneben ist und dazu wahrscheinlich noch von niemandem benutzt wurde. n0128/Physik lassen wir hingegen wie es ist. Was meinst du?

@acka47 acka47 mentioned this pull request Dec 3, 2024
@mic-men
Copy link
Contributor

mic-men commented Dec 6, 2024

Ich sehe das genauso. Vorschlag: Wir passen zumindest einmal n030010001 an zu n292, weil das ja komplett daneben ist und dazu wahrscheinlich noch von niemandem benutzt wurde. n0128/Physik lassen wir hingegen wie es ist.

Ja, dem kann ich zustimmen. Tatsächlich haben wir das Fachgebiet von n292 bei uns bereits benutzt, aber das können wir mit überschaubarem Aufwand manuell nachpflegen.

@acka47 acka47 linked an issue Dec 9, 2024 that may be closed by this pull request
@acka47
Copy link
Member

acka47 commented Dec 9, 2024

Sieht gut aus, auch im SkoHub build dieses Branches:

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.

+1

@acka47 acka47 requested a review from mic-men December 9, 2024 14:06
@acka47 acka47 assigned mic-men and unassigned maipet Dec 9, 2024
Copy link
Contributor

@mic-men mic-men left a comment

Choose a reason for hiding this comment

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

Die ID-Änderung wie besprochen ist in Ordnung. Zu den Py-Skripten kann ich kein kompetentes Feedback geben.

@acka47 acka47 mentioned this pull request Dec 18, 2024
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.

Update gemäß neuester Destatis-Version
4 participants