Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-26 Thread Even Rouault
Le jeudi 26 juin 2014 23:21:58, Jeremy Palmer a écrit : > > Looking at the ogresrijsonreader.cpp code I can see I assumed that a > > esriPolygon was just a OGC Polygon, so with outer ring first and then > > inner rings. Fixing the code to support multi polygons should basically > > be a matter of c

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-26 Thread Jeremy Palmer
> Looking at the ogresrijsonreader.cpp code I can see I assumed that a > esriPolygon was just a OGC Polygon, so with outer ring first and then inner > rings. Fixing the code to support multi polygons should basically be a matter > of calling OGRGeometryFactory::organizePolygons. Thanks Even. Any

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-26 Thread Even Rouault
Le lundi 23 juin 2014 03:32:32, Jeremy Palmer a écrit : > I'm querying an ArcGIS service obtain polygon data, however the service > seems to define Island multi-polygons objects as single polygons with > multiple exterior rings. This creates invalid OGC geometries and is > causing processing proble

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-24 Thread Jeremy Palmer
> My apology for confusion. > I guess, your request is valid then you may want to open ticket. No probs. Thanks I've raise a ticket here: http://trac.osgeo.org/gdal/ticket/5538 Cheers Jeremy This message contains information, which may be in confidence and may be subject to legal privilege. I

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-24 Thread Mateusz Łoskot
On 24 June 2014 01:01, Jeremy Palmer wrote: >> Might be. I just pointed out that OGR GeoJSON driver is dedicated to GeoJSON >> specifically, not any JSON-encoded format of geometry. > > But doesn't the OGR GeoJSON driver have specific Esri JSON code? > http://trac.osgeo.org/gdal/browser/trunk/gd

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-23 Thread Jeremy Palmer
> Might be. I just pointed out that OGR GeoJSON driver is dedicated to GeoJSON > specifically, not any JSON-encoded format of geometry. But doesn't the OGR GeoJSON driver have specific Esri JSON code? http://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/geojson/ogresrijsonreader.cpp Al

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-23 Thread Mateusz Łoskot
On 23 June 2014 19:46, Jeremy Palmer wrote: > http://resources.arcgis.com/en/help/arcgis-rest-api/index.html#//02r300n100 >> >> Can you? >> There is no geometry of multipolygon specified. >> AFAIU, for multipolygon ESRI REST produces a hybrid: a Polygon that >> contains detached rings >> (

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-23 Thread Jeremy Palmer
http://resources.arcgis.com/en/help/arcgis-rest-api/index.html#//02r300n100 > > Can you? > There is no geometry of multipolygon specified. > AFAIU, for multipolygon ESRI REST produces a hybrid: a Polygon that > contains detached rings > (or rings not in other rings, so they are not holes) a

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-23 Thread Jukka Rahkonen
Mateusz Łoskot loskot.net> writes: > > On 23 June 2014 03:32, Jeremy Palmer linz.govt.nz> wrote: > > > > Is this a bug in the data from the service, or just the way the Esri defines geometry objects (and therefor an > OGR bug)? > > AFAIU, this is not a bug in OGR, but OGR GeoJSON drivers works

Re: [gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-23 Thread Mateusz Łoskot
On 23 June 2014 03:32, Jeremy Palmer wrote: > > Is this a bug in the data from the service, or just the way the Esri defines > geometry objects (and therefor an OGR bug)? AFAIU, this is not a bug in OGR, but OGR GeoJSON drivers works with GeoJSON format whereas JSON produced by ESRI is a differe

[gdal-dev] ArcGIS RestAPI JSON Polygons

2014-06-22 Thread Jeremy Palmer
I'm querying an ArcGIS service obtain polygon data, however the service seems to define Island multi-polygons objects as single polygons with multiple exterior rings. This creates invalid OGC geometries and is causing processing problems in my workflow. He's the service and a validation test sh