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
