search

cityjson / specs

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

CityJSON 1.1 datasets using the "Generic" extension are broken #182

Closed clausnagel closed 1 year ago

clausnagel commented 1 year ago

With update to CityJSON 2.0 it seems that the "Generic" extension for CityJSON 1.1 is not available from cityjson.org anymore (see https://www.cityjson.org/extensions/) .

I have several CityJSON 1.1 datasets where the "Generic" extension version 1.0 is referenced via the URL https://www.cityjson.org/extensions/download/generic.ext.json. For example:

{
"type": "CityJSON",
"version":"1.1",
"extensions": {"Generic":{"url":"https://www.cityjson.org/extensions/download/generic.ext.json","version":"1.0"}}
}

Now the same URL serves the "Generic" extension version 1.0.1 which can only be used with CityJSON 2.0 but not with 1.1 anymore. So, all of these exisiting datasets are broken all of a sudden.

Of course, I can upgrade the datasets to CityJSON 2.0. But datasets should not just break because of a new CityJSON version. So, please consider serving extensions for different CityJSON versions under different URLs.

hugoledoux commented 1 year ago

ah yes that was bad, cheers for pointing it out.

I restored the v1.1 file at its original location and added a /v20/ folder where I put the new version (which is not useful, just a demo for people to understand how it works).

see website changes: https://github.com/cityjson/website/commit/5dc5b3d4b0c3654ed32791b90a91c0735866c113 will online in a few minutes

clausnagel commented 1 year ago

[like] Claus Nagel reacted to your message:

From: Hugo Ledoux @.> Sent: Friday, October 27, 2023 3:05:36 PM To: cityjson/specs @.> Cc: Claus Nagel @.>; Author @.> Subject: Re: [cityjson/specs] CityJSON 1.1 datasets using the "Generic" extension are broken (Issue #182)

ah yes that was bad, cheers for pointing it out.

I restored the v1.1 file at its original location and added a /v20/ folder where I put the new version (which is not useful, just a demo for people to understand how it works).

see website changes: @.***https://github.com/cityjson/website/commit/5dc5b3d4b0c3654ed32791b90a91c0735866c113 will online in a few minutes

— Reply to this email directly, view it on GitHubhttps://github.com/cityjson/specs/issues/182#issuecomment-1783073011, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABKDA74NCDKTPEIU43YT573YBPEUBAVCNFSM6AAAAAA6SPZH3CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGA3TGMBRGE. You are receiving this because you authored the thread.Message ID: @.***>