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

Ergänze fehlende LBZ-Notationen #6

Merged
merged 3 commits into from
Sep 11, 2024
Merged

Ergänze fehlende LBZ-Notationen #6

merged 3 commits into from
Sep 11, 2024

Conversation

acka47
Copy link
Contributor

@acka47 acka47 commented Aug 29, 2024

Resolves #4

@acka47
Copy link
Contributor Author

acka47 commented Aug 29, 2024

Ich habe endlich auch mal den Dateinamen für die LBZ-Notation en angepasst, damit das zumindest hier im Repo klar verständlich ist. Den Namensraum https://w3id.org/lobid/rpb2# werden wir ja – wie in hbz/lobid-vocabs#212 beschlossen –behalten.

@acka47
Copy link
Contributor Author

acka47 commented Aug 29, 2024

Die Ergänzungen lassen sich auch in SkoHub Vocabs begutachten unter https://skohub.io/hbz/lbz-vocabs/heads/4-addLbzNotations/w3id.org/lobid/rpb2.html.

@Merkler können Sie einmal drauschauen, ob das so passt?

@TobiasNx
Copy link
Contributor

TobiasNx commented Sep 4, 2024

Mir ist zudem aufgefallen, dass viele neue Labels für die Notationen Punkte anstelle von Kommas haben: z.B.

VORHER: Buch, Bibliothek, Information und Dokumentation

JETZT: Buch. Bibliothek. Information und Dokumentation

Ist das beabsichtigt?

@Merkler
Copy link
Contributor

Merkler commented Sep 4, 2024

Mir ist zudem aufgefallen, dass viele neue Labels für die Notationen Punkte anstelle von Kommas haben: z.B.

VORHER: Buch, Bibliothek, Information und Dokumentation

JETZT: Buch. Bibliothek. Information und Dokumentation

Ist das beabsichtigt?

Ich tippe mal, das ist "historisch gewachsen", stört aber auch erst einmal nicht - wenngleich es jetzt nicht einheitlich ist. Wenn das Vokabular final steht, können wir im LBZ in einer ruhigen Minute (nach unseren Großrpojekten) dies noch mal im Haus abstimmen und dann in github korrigieren.

@Merkler
Copy link
Contributor

Merkler commented Sep 4, 2024

Die Ergänzungen lassen sich auch in SkoHub Vocabs begutachten unter https://skohub.io/hbz/lbz-vocabs/heads/4-addLbzNotations/w3id.org/lobid/rpb2.html.

@Merkler können Sie einmal drauschauen, ob das so passt?

Habe draufgeschaut und mir ist beim Durchschauen auf den ersten Blick nur ein Fehler aufgefallen: Die 320 ist fälschlicherweise dublett drin (einmal unter der 300 und dort falsch sortiert, einmal auf oberster Ebene). Sie gehört unter die 300 in der richtigen Zahlensortierung.

Weitere Sortierungsfehler sind mir auf die Schnelle nicht aufgefallen, jedoch wäre hier noch ein kritisches drüberschauen sinnvoll.

@Merkler
Copy link
Contributor

Merkler commented Sep 4, 2024

@acka47

@TobiasNx TobiasNx assigned acka47 and unassigned Merkler Sep 4, 2024
@acka47
Copy link
Contributor Author

acka47 commented Sep 11, 2024

Die 320 ist fälschlicherweise dublett drin (einmal unter der 300 und dort falsch sortiert, einmal auf oberster Ebene). Sie gehört unter die 300 in der richtigen Zahlensortierung:

Ich habe mit 4e68587 die 320 aus der obersten Ebene entfernt und damit wird sie auch unter der 300 richtig einsortiert:

image

Ich merge den PR dann mal, wir können ja bei Bedarf weitere Anpassungen vornehmen.

@acka47 acka47 merged commit 03a5bc1 into main Sep 11, 2024
2 checks passed
blackwinter added a commit to hbz/limetrans that referenced this pull request Oct 1, 2024
hbz/lbz-vocabs#3; hbz/lbz-vocabs#6)"

This reverts commit 1038611.

Results in `java.lang.OutOfMemoryError thrown from the UncaughtExceptionHandler in thread "elasticsearch-rest-client-0-thread-[...]"`.
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.

Fehlende LBZ-Notationen ergänzen
3 participants