Peppe,

just a quick note: point d) was actually due to a bug in the RasterImageLayer class. It should be fixed now.

Alberto

On 08/06/2015 8:22 AM, Giuseppe Aruta wrote:
Hi Alberto,
I gave a look on this weekend to your plugin. Personally I support your idea to integrate your plugin into OpenJUMP. These are my impressions and some brief considerations:

a) Your raster styler (RS) has more advanced options than actual "Raster color editor" (RCE) integrated into OpenJUMP. Those avanced options can be summarized into 2 points:

  * RS has /ramp/, /user-defined intervals/ and /single/ values
    options while RCE supports basically only ramp. I started to
    integrate a possibility to have intervals on RCE but, looking at
    your code and plugin, I realized that you already made it (I was
    discovering the wheel again )
  *  RS provides a raster legend (what OJ currently need, in my
    opinion) while RCE doesn't.


b) I gave a look on your RS source code and I realize that many RS classes are already integrated into OpenJUMP

c) OpenJUMP RCE has some options that actually are missing on your RS (possibility to define an user range of values, invert color LUT), but those options are quite limited comparing to the advances of yur plugin (user-defined intervals and Legend, for instance)

d) In your RS there is an option to manage nodata value which seems not to working


As I wroye before, I vote to integrate your plugin into OpenJUMP, possibily as panel of Sextante Raster layer tree>Change Raster Raster Image properties, as you suggested some months ago, substituting RCE. In the future we can work to move the extra options in RCE (see point c) to your plugin and deactivate one. RCE will survive anyhow under Raster menu

Regarding point d), currently OpenJUMP handles nodata cells as transparent: no way to colourize them. This seems an advance for users: transparent->no data limit the area of analysis. On the other hand the possibility to colourized no-data cells with and "out-of-order" colour helps during some raster processes (for instance no data cells come out quickly if users apply Black/White colour schema and Red for no data). I would suggest to deactivate the nodata panel on your plugin until we find a solution.

Best Regards

Peppe


------------------------------------------------------------------------------


_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

------------------------------------------------------------------------------
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to