thx! and exactly my point :) ..ede On 27.01.2016 12:17, Alberto De Luca wrote: > Ede, > > yes I am aware of the majority of the additions relate to Layer.java, but I > wouldn't venture into commenting on those since my knowledge with regards > to the Layer class is limited. What I could say though is that for what > I've seen the design of Layer is very neat and though-through in comparison > to RasterImageLayer, therefore I would think more than twice before > touching it. > > Alberto > > On 27 January 2016 at 11:13, <edgar.sol...@web.de> wrote: > >> Alberto, >> >> you are aware most additions Peppe did were to Layer.java where they >> definitely don't belong? >> >> if it's needed in RasterImageLayer, i don't oppose at all. >> >> ..ede >> >> On 27.01.2016 10:53, Alberto De Luca wrote: >>> Hey guys, >>> >>> at the moment RasterImageLayer is quite a messy class (a mess to whom >> I've >>> been contributing, by the way), because there's a bit of everything in >> in, >>> from methods that manipulate the raster values to methods that take care >> of >>> the rendering on screen... >>> >>> So while we wait for the lucky ticket-owner to come along, I think that >> the >>> utility methods added by Peppe (that to me are definitely useful) should >>> stay: they just made the class a bit messier :P >>> >>> Alberto >>> >>> >>> On 27 January 2016 at 03:00, Stefan Steiniger <sst...@geo.uzh.ch> wrote: >>> >>>> see below >>>> >>>> Am 26.01.16 um 13:31 schrieb Giuseppe Aruta: >>>>> Just few extra details: >>>>> >>>>> 1) Temporary vector (Layer.class) and raster (RasterImageLayer.class) >>>>> layers (isTemporaryLayer()) >>>>> - Sextante is a different software and it handles raster and vector in >> a >>>>> separate way from OpenJUMP >>>>> - while TEMP files should be stored in Windows forever (except if user >>>>> does a deep OS cleaning) Linux and MacOSX delete these files from the >>>>> /TEMP folder on shuts down >>>>> Linux and Mac users should be aware that, if they shut down the OS and >>>>> if they don't save these temp data in another folder, they will lost >>>>> them. But I am sure that few know about this. >>>>> >>>>> Pratical usage: >>>>> Other temporary layers also layers with no datasource. >>>>> SaveLayrsWithoutDatasourcePlugIn of Michael already advises users about >>>>> the presence of these layers. >>>>> I am planning to expand this plugin to this other group of temporary >>>>> layers (the one stored into /TEMP folder) both Layers and >>>> RasterImageLayers. >>>>> I could work on the plugin without adding new boolean on System classes >>>>> (you are right, Ede, those functionality are already there) >>>>> But a flag like RasterImagerLayer.isTemporary() would avoid some extra >>>>> code and, more important, it could be used also in the future >>>> >>>> I agree with Peppe here... >>>> But perhaps we need to see how this fits also with Michaels DB >>>> experiences (and approaches). At least most Raster-Related helper >>>> classes make sense to me. Perhaps Alberto has a comment on that too. >>>> And if Ede points out to the original JUMP design/development... well >>>> the design focus was on vector... everything else raster wise was more >>>> experimental (my opinion). And this is now also some wooohhhoo 12-13 >>>> years ago... ;) >>>> >>>> Ideally of course one would need to do what QGIS did, a complete >>>> redesign of the core, but... (perhaps someone wins in the lottery and >>>> funds 5 devs from that :) >>>> >>>> cheers, >>>> Stefan >>>> >>>> >>>> >> ------------------------------------------------------------------------------ >>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance >>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month >>>> Monitor end-to-end web transactions and take corrective actions now >>>> Troubleshoot faster and improve end-user experience. Signup Now! >>>> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 >>>> _______________________________________________ >>>> Jump-pilot-devel mailing list >>>> Jump-pilot-devel@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>>> >>> >>> >>> >>> >> ------------------------------------------------------------------------------ >>> Site24x7 APM Insight: Get Deep Visibility into Application Performance >>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month >>> Monitor end-to-end web transactions and take corrective actions now >>> Troubleshoot faster and improve end-user experience. Signup Now! >>> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 >>> >>> >>> >>> _______________________________________________ >>> Jump-pilot-devel mailing list >>> Jump-pilot-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>> >> >> >> ------------------------------------------------------------------------------ >> Site24x7 APM Insight: Get Deep Visibility into Application Performance >> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month >> Monitor end-to-end web transactions and take corrective actions now >> Troubleshoot faster and improve end-user experience. Signup Now! >> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 >> _______________________________________________ >> Jump-pilot-devel mailing list >> Jump-pilot-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >> > > > > ------------------------------------------------------------------------------ > Site24x7 APM Insight: Get Deep Visibility into Application Performance > APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month > Monitor end-to-end web transactions and take corrective actions now > Troubleshoot faster and improve end-user experience. Signup Now! > http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 > > > > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >
------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
