search

opengeospatial / geoparquet

Specification for storing geospatial vector data (point, line, polygon) in Parquet
https://geoparquet.org
Apache License 2.0
837 stars 57 forks source link

Updated projjson schema link to 0.7 (and new link) #227

Closed cholmes closed 5 months ago

cholmes commented 5 months ago

Opening this PR more to continue the discussion in #180, since if we do want to be updating the schema versions than 1.1 release seems like a good time to.

I'm not sure what is the proper url to put in, I used https://proj.org/en/latest/schemas/v0.7/projjson.schema.json as the 'main' one seems to redirect there: https://proj.org/schemas/v0.7/projjson.schema.json But the 'id' is the latter one. There's additional discussion of this in https://github.com/OSGeo/PROJ/issues/4088

The case for updating the schema is that 0.5 -> 0.7 is mostly (all?) additive, so everything in the past should work.

The case against is that we should do something different than hard code in a version number. Or if we have to since that's how schemas work then we should target a new one.

jorisvandenbossche commented 5 months ago

I'm not sure what is the proper url to put in, I used https://proj.org/en/latest/schemas/v0.7/projjson.schema.json as the 'main' one seems to redirect there: https://proj.org/schemas/v0.7/projjson.schema.json But the 'id' is the latter one.

Given that the GDAL documentation about PROJJSON still uses the https://proj.org/schemas/v0.7/projjson.schema.json url, I would follow their lead?

rouault commented 5 months ago

Given that the GDAL documentation about PROJJSON still uses the https://proj.org/schemas/v0.7/projjson.schema.json url, I would follow their lead?

yes, the redirection to the en/latest/ URL is a ReadTheDocs hosting implementation detail

tschaub commented 5 months ago

Should this reference in the example metadata also be changed? https://github.com/opengeospatial/geoparquet/blob/a684d7a72f950f290e64f057d47932ae03e457c6/examples/example_metadata.json#L32