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 der Systematik auf den Stand von 2023 #28

Merged
merged 7 commits into from
Jan 8, 2024

Conversation

lummerland
Copy link
Contributor

Hier sind die Änderungen integriert, die in der aktuellen Veröffentlichung vom Januar 2023 (https://www.destatis.de/DE/Methoden/Klassifikationen/Bildung/studenten-pruefungsstatistik.html) enthalten sind. Außerdem auch die (umfangreicheren) vorhergehenden Änderungen vom 17.11.2021. Dazu finde ich allerdings gerade kein verlinkbares Dokument, ich habe das PDF bei mir lokal gespeichert :| Vielleicht kann das ja jemand nachrecherchieren?

Zwei Fächer sind gestrichen worden, ich habe sie als deprecated markiert. Ansonsten nur neue Fächer, Umbenennungen und Verschiebungen.

Betrifft Issue #14.

@lummerland
Copy link
Contributor Author

FYI @TobiasNx @acka47

@acka47 acka47 self-requested a review March 9, 2023 11:51
@acka47 acka47 self-assigned this Mar 9, 2023
@lummerland
Copy link
Contributor Author

@acka47 Ich hab Antwort von destatis bekommen. Die Fächersystematik für WS2020/2021 ist unter https://www.statistischebibliothek.de/mir/receive/DEHeft_mods_00135733 zu finden :)

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.

Ist aus meiner Sicht in Ordnung, danke!

@acka47
Copy link
Member

acka47 commented Dec 11, 2023

Ich habe mal draufgeschaut, allerdings ohne es 1-zu-1 mit der PDF-Version abzugleichen. Hier mein Bericht:

  • Das turtle ist valide.
  • Es sind nun 345 statt bisher 340 ConceptSchemes.

Allerdings meckert ein Build mit SkoHub Vocabs und, der Shacl-Check sagt viermal "Violation":

Note should be provided with a language tag.

(Siehe auch https://skohub.io/build/?id=fdfba9db-a905-4b46-a7a6-6a7c89af95cd)

Es gibt aber nur zwei skos:notes, bei denen tatsächlich das language tag fehlt (@sroertgen, du weißt bestimmt eher, warum er aber vier Violations nennt):

Wenn du das noch korrigierst, sollte es passen, @lummerland .

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.

@sroertgen
Copy link
Contributor

Es gibt aber nur zwei skos:notes, bei denen tatsächlich das language tag fehlt (@sroertgen, du weißt bestimmt eher, warum er aber vier Violations nennt):

Interessant. Schaue ich mir mal an!

@acka47
Copy link
Member

acka47 commented Dec 12, 2023

Im heutigen Treffen kam noch die Frage auf, was wir mit gelöschten/deprecated skos:Conceptss machen bzw. wie verarbeitende Systeme damit umgehen sollen. @mic-men, wir haben da ein bisschen aneinander vorbei geredet, weil es zwei verschiedene Fälle gibt.

1. deprecated: true

Es gibt zwei Fälle, in denen ein Konzept als deprecated markiert ist, siehe 7af142f

Michaels Frage ging in die Richtung, wie Software-Implementierungen (u.U auch SkoHub) damit umgehen sollen. Z.B. sollen diese Werte ja nicht in Formularen zur Auswahl gestellt werden.

2. Löschung/Ersetzung einer Konzept-URI

Wenn ich das richtig sehe, gibt es dann noch einen einen Fall, wo eine Konzept-URI verschwindet, weil sie durch eine neue ersetzt wird:

https://github.com/dini-ag-kim/hochschulfaechersystematik/pull/28/files#diff-7faa76cd7d77a0b2a88e90859a93a364d0eaad01aa2f7e4250690de7be1b5f55L595-R603

Den zweiten Fall können wir vermeiden, indem wir weiter die alte URI verwenden, die dann halt nicht mit der Notation übereinstimmt. (Entstanden ist das Problem glaube ich, als in diesem Commit nicht die URI mit aktualisiert wurde, sondern nur die Notation.) Ich verstehe, dass das gegen das ästhetische Empfinden verstößt, prinzipiell sollten wir aber dafür sorgen, dass URIs stabil bleiben.

@acka47
Copy link
Member

acka47 commented Dec 12, 2023

Einen anderen Punkt habe ich im heutigen Treffen auch angesprochen: Wir müssten uns bei den stattgefundenen Anpassungen und Ergänzungen von Labels auch um deren Übersetzungen kümmern (en, nl, uk) bzw. überlegen, wie wir damit umgehen.

@mic-men
Copy link
Contributor

mic-men commented Jan 5, 2024

Im heutigen Treffen kam noch die Frage auf, was wir mit gelöschten/deprecated skos:Conceptss machen bzw. wie verarbeitende Systeme damit umgehen sollen. @mic-men, wir haben da ein bisschen aneinander vorbei geredet, weil es zwei verschiedene Fälle gibt.

1. deprecated: true

Es gibt zwei Fälle, in denen ein Konzept als deprecated markiert ist, siehe 7af142f

Michaels Frage ging in die Richtung, wie Software-Implementierungen (u.U auch SkoHub) damit umgehen sollen. Z.B. sollen diese Werte ja nicht in Formularen zur Auswahl gestellt werden.

Genau, diesen Punkt würde ich gern besprechen. Sie dürfen nicht mehr auswählbar sein. Aber was ist mit vergebenen Werten? Sollen diese so bleiben oder auf andere abgebildet werden? Wenn letzteres, wie?

@lummerland
Copy link
Contributor Author

Ich habe mal draufgeschaut, allerdings ohne es 1-zu-1 mit der PDF-Version abzugleichen. Hier mein Bericht:

* Das turtle ist valide.

* Es sind nun 345 statt bisher 340 `ConceptScheme`s.

Allerdings meckert ein Build mit SkoHub Vocabs und, der Shacl-Check sagt viermal "Violation":

Note should be provided with a language tag.

(Siehe auch https://skohub.io/build/?id=fdfba9db-a905-4b46-a7a6-6a7c89af95cd)

Es gibt aber nur zwei skos:notes, bei denen tatsächlich das language tag fehlt (@sroertgen, du weißt bestimmt eher, warum er aber vier Violations nennt):

* https://github.com/dini-ag-kim/hochschulfaechersystematik/blob/7af142f0e4caf81e3c8979c65702269c0cdadfbd/hochschulfaechersystematik.ttl#L1103

* https://github.com/dini-ag-kim/hochschulfaechersystematik/blob/7af142f0e4caf81e3c8979c65702269c0cdadfbd/hochschulfaechersystematik.ttl#L1635

Wenn du das noch korrigierst, sollte es passen, @lummerland .

Die beiden Sprach-Tags sind integriert.

@lummerland
Copy link
Contributor Author

2. Löschung/Ersetzung einer Konzept-URI

Wenn ich das richtig sehe, gibt es dann noch einen einen Fall, wo eine Konzept-URI verschwindet, weil sie durch eine neue ersetzt wird:

https://github.com/dini-ag-kim/hochschulfaechersystematik/pull/28/files#diff-7faa76cd7d77a0b2a88e90859a93a364d0eaad01aa2f7e4250690de7be1b5f55L595-R603

Den zweiten Fall können wir vermeiden, indem wir weiter die alte URI verwenden, die dann halt nicht mit der Notation übereinstimmt. (Entstanden ist das Problem glaube ich, als in diesem Commit nicht die URI mit aktualisiert wurde, sondern nur die Notation.) Ich verstehe, dass das gegen das ästhetische Empfinden verstößt, prinzipiell sollten wir aber dafür sorgen, dass URIs stabil bleiben.

Ich habe die Änderung der URI bzw. des Concept-Namens wieder rückgängig gemacht. Ich hatte nicht darüber nachgedacht was diese Änderung bedeutet. Ist aber auch gar nicht so einfach, wenn man einerseits die Concept-Namen sinnvoll vergeben will (statt z.B. unlesbare UUIDs oder dergleichen zu verwenden) aber andererseits damit auch für lange Zeit feste URIs definiert :)

@acka47
Copy link
Member

acka47 commented Jan 8, 2024

Ist aber auch gar nicht so einfach, wenn man einerseits die Concept-Namen sinnvoll vergeben will (statt z.B. unlesbare UUIDs oder dergleichen zu verwenden) aber andererseits damit auch für lange Zeit feste URIs definiert :)

Ja, da hast du Recht. Das kann schnell zu Konflikten kommen. Passenderweise werden die Notationen der Destatis-Systematik offensichtlich auch nie geändert, auch wenn eine Systemstelle komplett umgehängt wird. Deshalb fehlt da mittlerweile jede Konsistenz, z.B.:

image

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.

Sieht alles gut aus. Ich habe jetzt noch zwei englische Labels vorgeschlagen für die neuen Concepts. Wenn du die noch ergänzt, gebe ich grünes Licht und merge das. (Hätte auch selbst commitet aber schaffe ich das nicht, weil der Branch nicht hier im Repo ist. Sollte eigentlich gehen, aber ich habe es jetzt aufgegeben. Wenn du den PR nächstes Mal von einem Branch aus diesem Repo aufmachst, wird das leichter.)

@acka47 acka47 assigned lummerland and unassigned acka47 Jan 8, 2024
lummerland and others added 3 commits January 8, 2024 11:38
Co-authored-by: Adrian Pohl <acka47@users.noreply.github.com>
Co-authored-by: Adrian Pohl <acka47@users.noreply.github.com>
@lummerland
Copy link
Contributor Author

@acka47 Ergänzungen sind drin, danke dir! Ja, ich weiß auch nicht, warum ich den Weg über den Fork gegangen bin. Werde ich aber nicht mehr tun, sorry :)

@acka47 acka47 merged commit f49c651 into dini-ag-kim:master Jan 8, 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.

4 participants