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

add documents of other administrative entities + json schema #147

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

saerdnaer
Copy link
Contributor

@saerdnaer saerdnaer commented Oct 31, 2021

Das wäre mein Vorschlag fürs JSON Format der Quell-Dokumente:

  {
    "entity": {
      "id": "DE913",
      "name": "Wolfsburg",
      "adminLevel": 6,
      "ags": "03103000",
      "wikidata": "https://www.wikidata.org/wiki/Q3014",
      "osm": "https://www.openstreetmap.org/relation/163046"
    },
    "documents": [
      {
        "title": "CO2-Bilanz und -Minderungskonzept der Stadt Wolfsburg (2.Auflage)",
        "url": "https://mycovenant.eumayors.eu/docs/seap/13939_1448983219.pdf",
        "type": "PLAN"
      },
      {
        "title": "CO2-Zwischenbericht 2011",
        "url": "https://mycovenant.eumayors.eu/docs/seap/13939_1448983406.pdf",
        "type": "REPORT"
      },
      {
        "title": "CO2-Zwischenbericht 2014",
        "url": "https://mycovenant.eumayors.eu/docs/seap/13939_1448983179.pdf",
        "type": "REPORT"
      },
      {
        "title": "CO2-Zwischenbericht 2016",
        "url": "https://mycovenant.eumayors.eu/docs/seap/13939_1461306674.pdf",
        "type": "REPORT"
      }
    ]
  },

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

Copy link
Collaborator

@jonek jonek left a 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.

data/documents-schema.json Outdated Show resolved Hide resolved
},
"required": ["data"]
},
"Entity": {
Copy link
Collaborator

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"?

Copy link
Contributor Author

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": {
Copy link
Collaborator

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.

Copy link
Contributor Author

@saerdnaer saerdnaer Oct 31, 2021

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

Suggested change
"adminLevel": {
"adminLevel": {
"description": "Landkreis: 6, Gemeinde: 7 etc. – vgl. https://wiki.openstreetmap.org/wiki/DE:Grenze#Innerstaatliche_Grenzen ",

Copy link
Contributor Author

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

"adminLevel": {
"type": "integer"
},
"ags": {
Copy link
Collaborator

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?

Copy link
Contributor Author

@saerdnaer saerdnaer Oct 31, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

@saerdnaer saerdnaer Oct 31, 2021

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.

data/documents-schema.json Show resolved Hide resolved
data/documents-schema.json Show resolved Hide resolved
@saerdnaer saerdnaer changed the title add documents of other administraive entities + json schema add documents of other administrative entities + json schema Oct 31, 2021
@saerdnaer saerdnaer requested a review from jonek October 3, 2022 11:04
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.

2 participants