Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Andrea Aime
Michael Bedward ha scritto: > Ah, I meant your new StyleBuilder Jody > > On 4 March 2010 12:57, Jody Garnett wrote: >> I had a brief chat with andrea. He has not interest in proceeding with >> StyleBuilder (I am going to look over it critically myself). The new StyleBuilder was originally develo

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Michael Bedward
Ah, I meant your new StyleBuilder Jody On 4 March 2010 12:57, Jody Garnett wrote: > I had a brief chat with andrea. He has not interest in proceeding with > StyleBuilder (I am going to look over it critically myself). --

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Jody Garnett
I had a brief chat with andrea. He has not interest in proceeding with StyleBuilder (I am going to look over it critically myself). I am going to split uDig way from the SLD class; leaving Andrea to change to returning float. Jody On Thu, Mar 4, 2010 at 11:56 AM, Michael Bedward wrote: > It wou

Re: [Geotools-devel] Release

2010-03-03 Thread Jody Garnett
Thank you - I will go ahead and announce then. Jody On Thu, Mar 4, 2010 at 10:24 AM, Michael Bedward wrote: > Hi Jody, > > Just tested the project.zip file and all went well. > > Michael > -- Download Intel® Parallel S

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Michael Bedward
It would be great to use the new StyleBuilder but there will still be a need for those methods in SLD that query Styles won't there ? In other words, should the plan be to pull out all of the building methods from SLD, once StyleBuilder is in business, and tidy up the remaining query methods ? Mi

Re: [Geotools-devel] Release

2010-03-03 Thread Michael Bedward
Hi Jody, Just tested the project.zip file and all went well. Michael -- Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for

[Geotools-devel] Hudson build is back to normal: geotools-trunk #2473

2010-03-03 Thread Hudson
See -- Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications f

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Jody Garnett
Okay next suggestion (helpful this time). uDig actually uses a subclass of SLD (called SLDs ... sigh). I will just copy the methods over and break that link; leaving GeoTools free to change. As I said I would like to migrate away from using this class. Now that still may not be a useful suggest

Re: [Geotools-devel] Build failed in Hudson: geotools-trunk #2472

2010-03-03 Thread Michael Bedward
My fault - sorry. Andrea, could you have a look at the latest version of the unit tests that I attached to GEOT-2960 please (EnvFunctionTest.java, attachment 3) I did some hundreds of replicate test runs on my machine and the errors seem to be cured by this version. cheers Michael -

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Michael Bedward
I'd vote to fix the offending methods at the expense of backwards compatibility. It seems unlikely that this will cause anything other than minor inconvenience for users. As an aside, I remember that the SLD class grew larger when the FOSS4G tutorials were being prepared and we're advertising it (

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Andrea Aime
Jody Garnett ha scritto: > So far uDig is the major user of the SLD class. I am trying to migrate away > from it to a StyleBuilder :-( > > Can you change the return type to a float (cannot remember which direction > automatic casting is allowed). > > int width = SLD.width( stroke ); // does th

Re: [Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Jody Garnett
So far uDig is the major user of the SLD class. I am trying to migrate away from it to a StyleBuilder :-( Can you change the return type to a float (cannot remember which direction automatic casting is allowed). int width = SLD.width( stroke ); // does this work when SLD returns a float? If n

Re: [Geotools-devel] Release

2010-03-03 Thread Jody Garnett
Release went smoothly today thus far... - tagged 2.6.2 - deployed 2.6.2 - jira released (Release notes after the break) - leaving my computer on uploading the files to source forge (If I can ask someone to test the project zip) Tomorrow we can update the downloads page (do we still do that?) and

[Geotools-devel] SLD utility vs SLDStyleFactory internal methods

2010-03-03 Thread Andrea Aime
Hi, today I've stumbled into a class that I did not know existed, the SLD utility class (well, I actually made a commit on that class three years ago but haven't used it since). The SLD utility has utility methods that return, for example, the width of a stroke. Unfortunately they are also broken,

[Geotools-devel] Build failed in Hudson: geotools-trunk #2472

2010-03-03 Thread Hudson
See Changes: [bencaradocdavies] Added type guards to prevent conflict between app-schema and wms jxpath NodePointerFactory implementations [GEOS-3525] -- [...truncated 964 lines...] Mar 3,