cityjson / specs

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

is no Semantic Surface Object, should it be ``{}`` or ``null`` #8

Closed hugoledoux closed 6 years ago

hugoledoux commented 6 years ago
"semantics": [
      {
        "type": "RoofSurface",
        "slope": 16.4,
        "solar-potential": 5
      },
      {},
      {
        "type": "GroundSurface"
      }
    ]

or null instead? Or both?

clausnagel commented 6 years ago

Only null in my opinion.

kenohori commented 6 years ago

I agree, null seems best.

hugoledoux commented 6 years ago

thanks all, I was looking at this and thought null should be there, you confirmed this.

But it seems that we our new proposal that won’t be necessary actually!

On Thu, Oct 12, 2017 at 4:24 AM, Ken Arroyo Ohori notifications@github.com wrote:

I agree, null seems best.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tudelft3d/cityjson/issues/8#issuecomment-336001716, or mute the thread https://github.com/notifications/unsubscribe-auth/ABeZFiw0ytxLCN6L9h881lkHOVk050R6ks5srXhngaJpZM4P0Mes .

hugoledoux commented 6 years ago

no necessary anymore, from v0.5 the Semantic Surface are enumerated and links to them provided.