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

addSkoselementsforNWBibMALISProjekt (#204) #205

Merged
merged 20 commits into from
Dec 6, 2024

Conversation

HMetzner
Copy link
Collaborator

@HMetzner HMetzner commented Nov 15, 2023

Resolves #204 Notations 100000-142370

@HMetzner
Copy link
Collaborator Author

Zu Testzwecken habe ich bei ein paar Notationen skos:exactMatch aus demThesaurus der ZBW gesetzt. Diese Links werden auch in der GND angezeigt.

@acka47
Copy link
Contributor

acka47 commented Nov 16, 2023

Hallo @HMetzner , wo ich lese "Zu Testzwecken": Wusstest du, dass auch einzelne git-Branches von SkoHub gebaut werden? Das heißt, deine Anpassungen im Branch 204-addSkoselementsforNWBibMALIS-Projekt sind hier einzusehen: https://skohub.io/hbz/lobid-vocabs/heads/204-addSkoselementsforNWBibMALIS-Projekt/nwbib.de/subjects.de.html

Ich glaube, es wäre sinnvoll, wenn du erstmal mehr in einem Branch arbeitest und dann am Ende einen größeren PR hast. So bist du unabhängig von uns und wir müssen nicht so viel PRs reviewen.

@acka47
Copy link
Contributor

acka47 commented Nov 16, 2023

Andererseits ist wahrscheinlich ein PR sinnvoll, wenn du – wie jetzt – neue Dinge ausprobierst. Du hast hier mit skos:exactMatch gearbeitet. Ich habe das jetzt nicht im einzelnen überprüft und bin selbst auch unerfahren, was die Vergabe von closeMatch vs. exactMatch angeht. Hast du dich damit einmal näher auseinandergesetzt? Gibt es da irgendwo einen best Practice Guide?

@HMetzner
Copy link
Collaborator Author

Hallo @acka47, hier sind skos:closeMatch und skos:exactMatch voneinander abgegrenzt: https://dini-ag-kim.github.io/skos-einfuehrung/#/skos-elemente.
Für die Bearbeitung finde ich es auch sinnvoller, weiter in diesen PR zu committen. Wenn ich neue Dinge ausprobiere, kann ich einen neuen PR machen.

@HMetzner
Copy link
Collaborator Author

Dann melde ich mich, wenn die Bearbeitung in diesem PR abgeschlossen ist. VG

@acka47
Copy link
Contributor

acka47 commented Nov 20, 2023

Dann melde ich mich, wenn die Bearbeitung in diesem PR abgeschlossen ist

Du kannst das auch zu einem Draft PR machen, siehe https://stackoverflow.com/questions/55070723/how-do-i-change-from-a-pr-to-a-draft-pr-at-github . Sobald du soweit bist, kannst du es wieder auf einen fertigen PR stellen und es mir zum Review zuweisen.

@HMetzner HMetzner marked this pull request as draft November 20, 2023 09:17
@HMetzner HMetzner marked this pull request as ready for review December 21, 2023 14:53
Copy link
Contributor

@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 schon sehr gut aus. Allerdings scheint mir an einigen Stellen skos:exactMatch nicht angebracht, wenn – z.B. bei N844040 "Burgen. Schlösser" oder bei N841070 "Künstler" – auf je zwei Begriffe aus WIkidata und/oder GND verwiesen wird, Dann matcht das Verwiesene ja nur zum Teil auf das NWBib-Konzept und es sollte skos:narrowMatch verwendet werden, oder?

nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
nwbib/nwbib.ttl Outdated Show resolved Hide resolved
@acka47 acka47 assigned HMetzner and unassigned acka47 Jan 5, 2024
@acka47
Copy link
Contributor

acka47 commented Feb 16, 2024

Hi @HMetzner , was ist hier der Status?

@HMetzner
Copy link
Collaborator Author

HMetzner commented Apr 8, 2024

Hey Adrian, hier ist der Stand, dass es noch offen ist. :) Ich halte skos:narrowMatch für nicht passend. Es ist die gleiche Bedeutung wie in Wikidata (nur eben in der deutschen GND mit weiblicher und männlicher Form). LG

@HMetzner HMetzner requested a review from acka47 July 4, 2024 12:17
Copy link
Contributor

@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.

Siehe meinen Kommentar zu skos:narrowMatch. ich bleibe dabei, dass das in den genannten Fällen anstatt skos:exactMatch benutzt werden sollte und verweise hier auf die analoge Nutzung im STW.

@HMetzner HMetzner requested a review from acka47 July 4, 2024 13:41
@acka47 acka47 removed their request for review July 4, 2024 14:09
@HMetzner HMetzner requested a review from acka47 July 5, 2024 14:07
@acka47 acka47 force-pushed the 204-addSkoselementsforNWBibMALIS-Projekt branch from dd6a5bb to 676782e Compare December 4, 2024 10:03
Copy link
Contributor

@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 merged commit 1d4fc90 into master Dec 6, 2024
2 checks passed
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.

Add Skos elements for NWBib (MALIS-Projekt)
2 participants