Re: [gdal-dev] ISO WKB

2013-12-13 Thread Paul Ramsey
Ah, right, it’s exposed.  Unfortunate, because it means that the OGC/ISO variant problem leaks into more external things than just WKB. I wonder how many people test dimensionality with getCoordinateDimension and how many do it with (wkb25DBit & getGeometryType()), no way to know unfortunately

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Even Rouault
Le vendredi 13 décembre 2013 21:51:57, Paul Ramsey a écrit : > Actually, I think changing the contract on getGeometryType to only ever > returning the basic (non-3d) type would clean up driver implementation > code a bit. Grepping for it in the drivers, you see a lot of > > CPLAssert(wkbFlatten(po

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Even Rouault
Le vendredi 13 décembre 2013 21:00:33, Paul Ramsey a écrit : > I’ve pushed up some work to do this > > https://github.com/pramsey/gdal/tree/isowkb > > I had to change *two* method signatures, which I don’t really like, both > exportToWkb and getGeometryType needed a variant parameter added. The >

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Paul Ramsey
Actually, I think changing the contract on getGeometryType to only ever returning the basic (non-3d) type would clean up driver implementation code a bit. Grepping for it in the drivers, you see a lot of CPLAssert(wkbFlatten(poGeom->getGeometryType()) == wkbPoint) and    || (poLine->getGeometr

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Paul Ramsey
I’ve pushed up some work to do this https://github.com/pramsey/gdal/tree/isowkb I had to change *two* method signatures, which I don’t really like, both exportToWkb and getGeometryType needed a variant parameter added. The problem is that getGeometryType returns the full WKB type, with dimensio

Re: [gdal-dev] [Qgis-developer] QGIS Multi-threaded Rendering

2013-12-13 Thread Even Rouault
Le vendredi 13 décembre 2013 08:25:55, Dmitriy Baryshnikov a écrit : > Hi, > > Why not reading small chunks (i.e. 256 x 256) with RasterIO > 3> from big raster? We'll get less memory usage. I think QGIS currently does a

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Pepijn Van Eeckhoudt
On 13-12-13 00:24, Paul Ramsey wrote: The GeoPackage document actually specifies a representation for POINT EMPTY, "In GeoPackages these points SHALL be encoded as a Point where each coordinate value is set to an IEEE-754 quiet NaN value". I'm to blame for this one. It's specified in the GeoPac

Re: [gdal-dev] ISO WKB

2013-12-13 Thread Mateusz Loskot
On 12 December 2013 23:24, Paul Ramsey wrote: > > The GeoPackage document actually specifies a representation for POINT EMPTY, > "In GeoPackages these points SHALL be encoded as a Point where each > coordinate value is set to an IEEE-754 quiet NaN value”. Couldn't OGC and ISO unify *all* their sp