Sveino / Nordic44

Synthetic Power System of the Nordic high voltage transmission System
Other
1 stars 0 forks source link

problems with Nordic44-30-HV1_GL.geojson #11

Open VladimirAlexiev opened 2 years ago

VladimirAlexiev commented 2 years ago

Problems:


Most importantly: you use https://github.com/geojson/geojson-ld to attach geo regions to assets, eg:

<urn:uuid:1c46e73d-6333-48ac-9367-1610caf1d3df>
        cim:IdentifiedObject.description
                "Right-of-Way Kristiansand Arendal" ;
        cim:IdentifiedObject.name  "RoW 300KRISTIAN-ARENDAL" .

<urn:uuid:ce9450ba-f145-4a83-bcbb-cc9098f4fa23>
        rdf:type            geojson:Feature ;
        geojson:geometry    [ rdf:type geojson:MultiPolygon ; geojson:coordinates (list of lists)
        geojson:properties  [ nc:RightOfWay  <urn:uuid:1c46e73d-6333-48ac-9367-1610caf1d3df> ] .

There are numerous problems with this approach:

Sveino commented 1 year ago
  • name it .jsonld rather than .geojson because it's first and foremost JSONLD, even though it also carries geo info

The primary reason for using .geojson is so that VSCode can do automatic validation against GeoJSON. In addtion Github does display the map information with this name. I would in general say that this is primarly a GeoJSON were we do use JSON-LD rather than JSON.

  • the prop <dcterm:created> is misspelt: you've defined prefix dcterms. However, I strongly suggest you switch to the shorter dct, which is more commonly used

https://www.w3.org/TR/vocab-dcat-3/ are using dcterms, so the thinking was to be alined with latest and greatest.

  • dcterms:LocationPeriodOrJurisdictionis a class not a property (the property is dct:coverage). But you better use the more specific propertydct:spatial`

So the thinking is that dct:spartial is not necessary correct. Since we are trying to refer to the area of responebility. Statnett is responseble to model the transmission network, that can be spartial overlapping with distribution. But do you think it would be OK to use dct:spartial?

Most importantly: you use https://github.com/geojson/geojson-ld to attach geo regions to assets, eg:

This is a good alternative, but the point was to be confom to GeoJSON. This does not prevent us from adding geojson-ld to the asset. The GeoJSON export is created for the export from Geographical Information System (GIS).

  • this is a non-standard way by "Sean Gillies" and a few other guys. It's not an OGC standard.

The proposal is created based on examples from OGC. I have also send an email to Nicholas Car for comments. It would be really good if you can make an example that would conferm to GeoJSON.

VladimirAlexiev commented 1 year ago

Please read more carefully what I wrote: <dcterm:created> uses a misspelt prefix, so it's converted to a full URL with invalid scheme dcterm: rather than a prefixed URL with dcterms:.

There are two kinds of coverage: spatial, temporal. Norway is related to spatial, not temporal.

GeoJSON-LD is not any standard.

I could make an example and continue with advice if we get into some engagement.