cityjson / specs

Specifications for CityJSON, a JSON-based encoding for 3D city models
https://cityjson.org
Creative Commons Zero v1.0 Universal
108 stars 25 forks source link

Add support for compound CRS as CURIEs #193

Open balazsdukai opened 7 months ago

balazsdukai commented 7 months ago

Allow compound CRS in metadata.referenceSystem the form of CURIEs, in addition to the current URI syntax.

For example:

"referenceSystem": [
    "[EPSG:4326]",
    "[EPSG:3855]"
  ]

The schema allows any UPPER/lowercase letter or digit as identifier, and any UPPER/lowercase letter as authority.

However, this change breaks backwards compatibility, so it would require a new major version.

If we decide to merge this PR, the specs need to be updated too.

Fixes #181

hugoledoux commented 1 month ago

This would break v2.0 as not backwards compatible... One way to fix this would be to add a new property "referenceSystem3D", but I am not sure if compound have only 2 components max. You know?

balazsdukai commented 1 month ago

My interpretation of the specs is that there are only two.

But I would rather keep this for a next major version than having a "referenceSystem3D" tbh. I think there should be one, and only one place to declare sth, and two referenceSystem-s would break that.