Re: [Geotools-devel] Raster classifier operation for large inputs

2018-11-14 Thread Justin Deoliveira
Ahhh sorry, totally missed that, my apologies, too much multitasking on my part :) Anyways, +1 from me fwiw. On Wed, Nov 14, 2018 at 11:59 AM Andrea Aime wrote: > On Wed, Nov 14, 2018 at 7:54 PM Justin Deoliveira > wrote: > >> Hey Andrea, >> All of your changes sound good to me. Only question I

Re: [Geotools-devel] Raster classifier operation for large inputs

2018-11-14 Thread Andrea Aime
On Wed, Nov 14, 2018 at 7:54 PM Justin Deoliveira wrote: > Hey Andrea, > All of your changes sound good to me. Only question I have is whether your > proposed change will replace what is there? Or is your thought to add some > config parameter that would trigger the histogram / approximation base

Re: [Geotools-devel] Raster classifier operation for large inputs

2018-11-14 Thread Simone Giannecchini
For what it's worth, I like the plan. Improving current code, making it shareable between multiple parts of the codebase by pushing back to JAI-Ext It will be nice to have in the SLDService. Regards, Simone Giannecchini == GeoServer Professional Services from the experts! Visit http://goo.gl/

Re: [Geotools-devel] Raster classifier operation for large inputs

2018-11-14 Thread Justin Deoliveira
Hey Andrea, All of your changes sound good to me. Only question I have is whether your proposed change will replace what is there? Or is your thought to add some config parameter that would trigger the histogram / approximation based method? As for moving the code to jai-text definitely makes sen

[Geotools-devel] Raster classifier operation for large inputs

2018-11-14 Thread Andrea Aime
Hi, I'm looking into extending the GeoServer SLDService API to work against raster data too. The current code in that module works off the vector classification functions for equal intervals, natural breaks and quantiles. When looking at extending it for rasters, I stumbled into the ClassBreaksOpI