<div dir="ltr">Hi Even,<div><br></div><div>All the above (or below) sounds good to me. Could a "description" member be added as well?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Nov 19, 2016 at 4:24 PM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
Currently the GeoJSON datasource always return "OGRGeoJSON" as a layer name,<br>
which is quite boring (if you convert the geojson into a multi-layer capable<br>
dataousrce, you always have to rename the layer to something more meaningful)<br>
<br>
I'd consider changing that to take the basename of the datasource, ie "my" if<br>
the filename is "/path/to/my.geojson". This would be similar to many other<br>
drivers like Shapefile, CSV, MapInfo File, JML ... (ie drivers with a single<br>
layer per file)<br>
<br>
For URLS coming from HTTP services with query parameters "?foo=bar&...", we<br>
would likely keep the "OGRGeoJSON" name.<br>
<br>
I'd also consider adding and reading a "name" extension member in the<br>
FeatureCollection object to store/retrieve the layer name. This would have the<br>
priority over the filename based rule when it exists.<br>
<br>
{<br>
        "type": "FeatureCollection",<br>
        "name": "layer_name",<br>
        "features": [<br>
        }<br>
}<br>
<br>
Thoughts ?<br>
<span class="HOEnZb"><font color="#888888"><br>
Even<br>
<br>
--<br>
Spatialys - Geospatial professional services<br>
<a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
______________________________<wbr>_________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a></font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Sean Gillies</div></div>
</div>