Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Martin Desruisseaux
Simone Giannecchini a écrit : > Well, I see that you never fought with the reversed-axis problem but > believe me when everybody understands lon,lat while our crs where > mostly lat,lon these methods come pretty hand to reverse things .-). I never fought the reversed-axis problem in the sense that

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Martin Desruisseaux
Simone Giannecchini a écrit : > First of all an annotation. I am currently working exclusively on > 2.3.x, I will port all the fixes I am making to trunk not before the > end f the month. Which version are you working on? On the trunk. But they are all changes that I would like to port to the 2.3

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Simone Giannecchini
First of all an annotation. I am currently working exclusively on 2.3.x, I will port all the fixes I am making to trunk not before the end f the month. Which version are you working on? On 11/12/06, Martin Desruisseaux <[EMAIL PROTECTED]> wrote: > Hello Simone > > Thanks for your feedback. > > Sim

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Martin Desruisseaux
Martin Desruisseaux a écrit : > I can move them in an utility class then (will try to find which utility > class > could be appropriate). But actually the only usage I found right now was in > RendererUtilities, and it was possible to avoid it by constructing a slightly > offseted envelope. I

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Martin Desruisseaux
Hello Simone Thanks for your feedback. Simone Giannecchini a écrit : > Well, in this is the sentence "by definition" may have a different > meaning for different people because it is at the same time subjective > as well as dependent on the data source. Some examples, in GTOPO30 the > nodata valu

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-12 Thread Simone Giannecchini
On 11/11/06, Martin Desruisseaux <[EMAIL PROTECTED]> wrote: > I'm very late, but I finally started to review the coverage branch merge. I > would like to hold on the 2.3 release until I finished this review, since I > have > some questions. > > org.geotools.coverage.CategoryList >

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-11 Thread Martin Desruisseaux
Hello Simone Martin Desruisseaux a écrit : > Martin Desruisseaux a écrit : >> org.geotools.coverage.grid.GeneralGridGeometry >> -- >> [...snip...] >> >> - reverse >> - swapXY >> - getEnvelope(GridRange, MathTransform, CoordinateReferenceSystem, boo

Re: [Geotools-devel] Coverage branch merge: first questions

2006-11-11 Thread Martin Desruisseaux
Martin Desruisseaux a écrit : > org.geotools.coverage.grid.GeneralGridGeometry > -- > [...snip...] > > - reverse > - swapXY > - getEnvelope(GridRange, MathTransform, CoordinateReferenceSystem, boolean) > - getTransform(GridRange, Envelope, boolea

[Geotools-devel] Coverage branch merge: first questions

2006-11-11 Thread Martin Desruisseaux
I'm very late, but I finally started to review the coverage branch merge. I would like to hold on the 2.3 release until I finished this review, since I have some questions. org.geotools.coverage.CategoryList -- The implementation before merge was looking for a "n