Re: [Geotools-devel] Arcs, GML support

2014-07-11 Thread Andrea Aime
On Sat, Jul 12, 2014 at 2:12 AM, Jody Garnett wrote: > Had a review, lots of small feedback nothing that should get in your way. > > Example: with the base classes moved to the gt-api module the utility > method CurvedGeometries.isCurved utility method to the JTS utility class. > Eh, the problem

Re: [Geotools-devel] [Pull request #501] DB Idle connections improvements

2014-07-11 Thread Jody Garnett
Merged onto geotools master, still hoping to see some docs for the geoserver project (when you get a chance). Jody Garnett On Wed, Jul 9, 2014 at 11:04 AM, Jody Garnett wrote: > Excellent I will look forward to your revised pull request, and thanks for > escalating this on the devel list. > --

Re: [Geotools-devel] Arcs, GML support

2014-07-11 Thread Jody Garnett
Had a review, lots of small feedback nothing that should get in your way. Example: with the base classes moved to the gt-api module the utility method CurvedGeometries.isCurved utility method to the JTS utility class. Is there any reason we should not make this class the global default for JTSFac

[Geotools-devel] [jira] (GEOT-4845) BridjOGRDataStoreTest test failure on OSX (GDAL 1.9.2)

2014-07-11 Thread Jody Garnett (JIRA)
Title: Message Title Jody Garnett created an issue

Re: [Geotools-devel] library/metadata logging test failure?

2014-07-11 Thread Justin Deoliveira
Figured this out. I had installed the GDAL osx framework as part of running qgis and it dumped a bunch of jars into my jre extensions directory! Including commons-logging which was what was throwing things off. On Wed, Jul 9, 2014 at 2:01 PM, Jody Garnett wrote: > Trying to think what else coul

Re: [Geotools-devel] Arcs, GML support

2014-07-11 Thread Jody Garnett
Excellent work Andrea: and yeah it makes sense to move this geometry stiff to gt-api. Jody Garnett On Fri, Jul 11, 2014 at 4:04 AM, Andrea Aime wrote: > Hi, > the pull request for arcs now contains support for GML encoding > of curved geometries. > > I basically took what was already there and

[Geotools-devel] Arcs, GML support

2014-07-11 Thread Andrea Aime
Hi, the pull request for arcs now contains support for GML encoding of curved geometries. I basically took what was already there and made some changes: * curve support, for the part that I could work on (curves, rings and composite curves), is enabled by default now, making geometries with curv

[Geotools-devel] App-schema, xlinks, and curves

2014-07-11 Thread Andrea Aime
Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometry