funkyfuture / m4p0-rs-metadata-import

GNU Affero General Public License v3.0
0 stars 0 forks source link

Adds the SPECIFICATION.md #7

Closed funkyfuture closed 4 years ago

funkyfuture commented 5 years ago

irgendwie habe ich den eindruck, dass das langsam rund wird.

fdiehr commented 5 years ago

hier nochmal die aktualisierte Tabelle wie sie an die Teilprojekte rausgehen soll zum Testen. Kannst du da nochmal einen Blick drauf werfen? TEST_M5-ImportDigitaleObjekte.xlsx

Ich habe jetzt auch doch noch ein anderes Teilprojekt auserkoren, uns Testdaten zuschicken. Die sind immer sehr verlässlich und haben vor allem alle Medientypen bei sich im Projekt vertreten!

funkyfuture commented 5 years ago

ich hab nochmal einen commit zur vereinheitlichung nachgeschoben. schau mal vor allem, ob die obligatorisch/optional-angaben bei den 3d-sets hinhauen.

funkyfuture commented 5 years ago

zum excel-dokument:

generell hätte ich gerne maschinenfreundlichhere spaltentitel. ist es nicht möglich, in excel zeilen in der anzeige zu verstecken oder zu sperren, so dass die ersten drei zeilen könnten:

ansonsten bleibt hier die frage nach den arbriträren feldern zum beschreiben eines MO. du hast ja da die möglichekit gegeben, einen dateinamen anzugeben. daraus schließe ich, dass du die liefernden also json-dateien erstellen ließest. dann sollen sie das doch, in einer datei für alle bezugsentitäten, in der als top-level-schlüssel eben auch die Bezugsentität verwendet wird.

funkyfuture commented 5 years ago

angabe der optionalität von spalten vereinhetlichen

ah, peile jetzt den asterisken. is vielleicht zu dezent.

fdiehr commented 5 years ago

so, jetzt hier nochmal die Tabelle. Ich hoffe, es passt jetzt so? TEST_M5-ImportDigitaleObjekte.xlsx

Ich hatte wegen der json-Datei n Denkfehler drin vorhin. Ist natürlich quatsch. Klaro liefern sie ne Datei mit allen Objekten drin! Es wird allerdings keine JSON sein, sondern ne CSV, aber die lässt sich ja sehr easy konvertieren. Das wäre eine schöne erste Python Übung für mich würde ich meinen :)

funkyfuture commented 5 years ago

Das wäre eine schöne erste Python Übung für mich würde ich meinen

es macht nicht wirklich sinn, das in einem eigenen skript vorzuprozessieren. letztlich muss das serialisierte JSON ja in die SPARQL-Querys mit eingefügt werden. aber ja, ich kann den teil frei lassen und du implementierst das dann.

zur tabelle: mit diesen spaltennamen lässt sich locker arbeiten und die hinweise sind nme nachvollziehbar.

funkyfuture commented 5 years ago

okay, abgesehen von der komplementierenden ergänzung der ontologie (RDFGraph) würde ich sagen, dass dieses dokument hier erstmal ein ergebnis ist und würde das mergen. spricht deinerseits etwas dagegen?

alle künftigen änderungen würde ich dann gerne in separaten issues / PRs tracken wollen, um dann die womöglichen änderungen am code auch gut im blick behalten zu können.

fdiehr commented 5 years ago

make it so! und ich schick die tabelle an das teilprojekt!

funkyfuture commented 5 years ago

squashed and merged with 07c40cd39fd4d53299204d0d05585bd6aaef6128

funkyfuture commented 4 years ago

es macht keinen sinn an einem pull request, der bereits gemerged wurde, rumzuwurschteln.

es wäre sehr hilfreich, wenn du einen pull request mit deinen änderungswünschen öffnen würdest.