Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-18 Thread Andrea Aime
Andrea Aime ha scritto: Hi, after quite a bit of testing on our own and a cross review in person during the FOSS4G-IT 2009 conference in Lugano me and Simone have agreed it's time to turn the patch into committed status. Oh, just noticed the GeoTools patch was incomplete. Here is a better one.

Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-17 Thread Jody Garnett
That may of just answered my question Andrea. We make use of GridCoverageRenderer directly - but we start by handing it the buffer to draw into. If the main advantage is the fact that it can produce the final rendered image I will have to check if we can make use of that or not. Cheers, Jody

Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-16 Thread Andrea Aime
Jody Garnett ha scritto: > Not copying exactly; we construct a buffered image around a block of > memory already handled by SWT. And then let the java API draw directly > into it ... so I agree it should be a speed up. It will be a speed up only if you can take the RenderedImage coming out of JA

Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-16 Thread Andrea Aime
Michael Bedward ha scritto: > On 17 February 2010 11:02, Jody Garnett wrote:> > >> +1 (although I would request an IRC chat when making the transition please) >> > > Ditto for me. Ok, when do you want to chat about it? I was about to write a long mail explaining why you won't get any speedup in

Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-16 Thread Michael Bedward
On 17 February 2010 11:02, Jody Garnett wrote:> > +1 (although I would request an IRC chat when making the transition please) > Ditto for me. Thanks for this work Andrea and Simone ! Michael -- SOLARIS 10 is the OS for

Re: [Geotools-devel] Coverage speedup patch ready to be applied

2010-02-16 Thread Jody Garnett
> Performance and scalability wise we have seen very good > results, though Gabriel reported a slowdown in the specific > case of SDE. That slowdown has not been analyzed unfortunately > and we have no clue on how and why this happened. > We hope that by applying the patches we'll get more > feedba