Hi Alberto, sorry for the late answer. Thanks for fixing that bug. I also see that your plugin has a "restore original colours" option that works fine. Peppe
2015-06-08 12:04 GMT+02:00 Alberto De Luca - GeA < alberto.del...@geomaticaeambiente.com>: > 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 > listJump-pilot-devel@lists.sourceforge.nethttps://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 > >
------------------------------------------------------------------------------
_______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
