-
Notifications
You must be signed in to change notification settings - Fork 19
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
add documents of other administrative entities + json schema #147
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich glaube, du hattes auf dem HackDay in Moers angesprochen, ob wir die properties in JSON auf Englisch oder Deutsch (bzw. in der "Domänensprache") benennen wollen.
Jetzt hast du Englisch zur Beszeichnung der Properties genommen, aber Deutsch für die title
bzw. description
Properties im JSON-Schema.
Ich wäre hier auch an einem Meinungsbild aus dem Rest des Teams interesiert.
}, | ||
"required": ["data"] | ||
}, | ||
"Entity": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ist "Entity" hier nicht etwas sehr generisch? Wie wäre es mit "ReportingEntity" oder "RegionalAuthority" oder "ReportingRegionalAuthority"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja, weitere Alternativen wäre AdminUnit
, AdministrativeUnit
, Region
oder Area
... Such dir einfach das aus was dir am besten gefällt :-)
"name": { | ||
"type": "string" | ||
}, | ||
"adminLevel": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was bezeichnet adminLevel
hier? Einen NUTS3 oder LAU Code? Ich denke, wir sollten hier eine Beschreibung des Properties als description
oder title
hinzufügen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Die Idee war das man den Daten direkt ansieht ob sie sich auf einen Landkreis, Stadt(-teil/bezirtk), Gemeine usw. beziehen. Ich hatte ihr einfach den Wert aus OSM übernommen, vgl. https://wiki.openstreetmap.org/wiki/Key:admin_level bzw. https://wiki.openstreetmap.org/wiki/DE:Grenze#Innerstaatliche_Grenzen
"adminLevel": { | |
"adminLevel": { | |
"description": "Landkreis: 6, Gemeinde: 7 etc. – vgl. https://wiki.openstreetmap.org/wiki/DE:Grenze#Innerstaatliche_Grenzen ", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oder wir machen nen Enum draus...
data/documents-schema.json
Outdated
"adminLevel": { | ||
"type": "integer" | ||
}, | ||
"ags": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was bezeichnet das Property ags
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Den Amtlichen Gemeindeschlüssel, also das Wikidata Property https://www.wikidata.org/wiki/Property:P439 – wir sollten vielleicht auch noch den ARS (Amtlicher Regionalschlüssel) mit aufnehmen, vgl. https://openknowledgegermany.slack.com/archives/CQC9RQ1NU/p1635673845027400?thread_ts=1635672345.022800&cid=CQC9RQ1NU allerdings fehlt der aktuell noch in Wikidata...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://de.wikipedia.org/wiki/Amtlicher_Gemeindeschlüssel#Regionalschlüssel
Außerdem gibt es den zwölfstelligen Amtlichen Regionalschlüssel (ARS)[4], ursprünglich Regionalschlüssel (RS), der 1993/1994 eingeführt wurde und der ähnlich aufgebaut ist, in den aber im Vergleich zum AGS vor den letzten drei Ziffern vier Stellen zur Verschlüsselung der Gemeindeverbände eingefügt sind.[7][8] Der AGS ist also in den ARS integriert. Angaben auf der Gemeindeverbandsebene (in der EU: LAU 1-Ebene) werden zunehmend für den innereuropäischen und internationalen Vergleich von Eurostat benötigt und von den Statistischen Ämtern des Bundes und der Länder eingefordert. Die Amtliche Statistik in Deutschland strebt langfristig die Ablösung des achtstelligen AGS durch den zwölfstelligen Regionalschlüssel an.
Okay, ich ging zu dem Zeitpunkt noch davon aus das AGS und ARS enger verwandt sind... können wir dann auch einfach rauslassen und statt dessen nur den ARS aufführen.
Das wäre mein Vorschlag fürs JSON Format der Quell-Dokumente:
source: https://pad.okfn.de/p/klimaschutzpl%C3%A4ne via https://openknowledgegermany.slack.com/archives/CQC9RQ1NU/p1635599621014600
Wir könnten natürlich auch noch gleich nen Github-Action hinzufügen die das auch gleich validiert...