Re: [Geotools-devel] GEOT-4319, adding support for loading of gridshift files with or without the ClasspathGridShiftLocator

2019-08-02 Thread Brandon Brown
Hello Geotools Devs! I’ve created https://github.com/geotools/geotools/pull/2497 as a potential solution for this issue. I’ve tested this with a project I’m using at work and it does infact properly locate gridshift files from the classpath. I’d love any feedback on how to improve this and am

Re: [Geotools-devel] Supporting GDAL Virtual File Systems

2019-08-02 Thread Jody Garnett
I would recommend splitting the code into two - one for imageio-ext and one for geotools (so if you like you could set up the modules and package structure ahead of time. They have different licenses so any code going from ImageIO to geotools is fine (keep original Apache header and add GeoTools LG

Re: [Geotools-devel] Supporting GDAL Virtual File Systems

2019-08-02 Thread Josh Fix via GeoTools-Devel
Hey thanks Jody! Ya I would love to contribute it. I just figured it would be better to let it incubate in my repo until it's received enough criticism. I will definitely need my hand held a little as I've never contributed significant amounts of code to another project before. There are also s

Re: [Geotools-devel] Supporting GDAL Virtual File Systems

2019-08-02 Thread Jody Garnett
That is great josh! Is the goal to contribute this to imageio-ext? If you can try and cite your sources in the header it will help .. On Fri, Aug 2, 2019 at 1:03 PM Josh Fix via GeoTools-Devel < geotools-devel@lists.sourceforge.net> wrote: > So here is my first stab at the reader that supports V

Re: [Geotools-devel] Supporting GDAL Virtual File Systems

2019-08-02 Thread Josh Fix via GeoTools-Devel
So here is my first stab at the reader that supports VFS paths: https://github.com/joshfix/gdal-vfs-reader It is very much a prototype/WIP. I would guess that 95% of the code was simply copied and pasted from various sources in imageio-ext and geotools and reassembled into these new classes. I'

Re: [Geotools-devel] Include jdbc-hana in GeoTools 22.0 release

2019-08-02 Thread Ian Turton
+0 I've never heard of this DB so I can't comment Ian On Wed, 31 Jul 2019 at 18:57, Jody Garnett wrote: > The community module jdbc-hana is ready to be a geotools plugin with > Stefan as module maintainer. While I found some duplicate functionality and > opportunities for improvement the code h

Re: [Geotools-devel] Include jdbc-hana in GeoTools 22.0 release

2019-08-02 Thread Nuno Oliveira
+1 On Wed, 2019-07-31 at 10:56 -0700, Jody Garnett wrote: > The community module jdbc-hana is ready to be a geotools plugin with > Stefan as module maintainer. While I found some duplicate > functionality and opportunities for improvement the code had no IP > troubles, has appropriate documentation