search

georust / geozero

Zero-Copy reading and writing of geospatial data.
Apache License 2.0
336 stars 33 forks source link

geojson: should omit `name` field when dataset name is None #27

Closed michaelkirk closed 2 years ago

michaelkirk commented 2 years ago

Is there any value in including the blank name property?

I think we should do:

{ 
  "type": "FeatureCollection",
- "name": "",
  "features": [ ...]
}
pka commented 2 years ago

I can't remember any good reason for a blank name, so fine with omitting.

michaelkirk commented 2 years ago

Ok, I'll follow up with a PR momentarily.