Re: [Geotools-devel] handling of geometry crs

2010-07-29 Thread Rini.Angreani
2010 9:04 PM To: Angreani, Rini (CESRE, Kensington) Cc: jdeol...@opengeo.org; aa...@opengeo.org; geotools-devel@lists.sourceforge.net Subject: Re: [Geotools-devel] handling of geometry crs Not sure that will work (leaving it to us) keep the conversation going. Reni what are you expecting the ou

Re: [Geotools-devel] handling of geometry crs

2010-07-29 Thread Jody Garnett
> > -Original Message- > From: Justin Deoliveira [mailto:jdeol...@opengeo.org] > Sent: Wednesday, 28 July 2010 12:07 AM > To: Andrea Aime > Cc: Angreani, Rini (CESRE, Kensington); jody.garn...@gmail.com; > geotools-devel@lists.sourceforge.net > Subject: Re: [Geotools

Re: [Geotools-devel] handling of geometry crs

2010-07-29 Thread Rini.Angreani
nesday, 28 July 2010 12:07 AM To: Andrea Aime Cc: Angreani, Rini (CESRE, Kensington); jody.garn...@gmail.com; geotools-devel@lists.sourceforge.net Subject: Re: [Geotools-devel] handling of geometry crs On 10-07-27 3:25 AM, Andrea Aime wrote: > rini.angre...@csiro.au wrote: >> Hi Justin,

Re: [Geotools-devel] handling of geometry crs

2010-07-27 Thread Justin Deoliveira
On 10-07-27 3:25 AM, Andrea Aime wrote: > rini.angre...@csiro.au wrote: >> Hi Justin, >> >> I like your second idea, and it is actually similar to the patch Jody >> whipped up >> (http://jira.codehaus.org/secure/attachment/50280/GML2EncodingUtils.patch). >> >> Since AbstractFeatureTypeBinding.getPr

Re: [Geotools-devel] handling of geometry crs

2010-07-27 Thread Jody Garnett
.org] > Sent: Tuesday, 27 July 2010 12:32 AM > To: Andrea Aime > Cc: Angreani, Rini (CESRE, Kensington); jody.garn...@gmail.com; > geotools-devel@lists.sourceforge.net > Subject: Re: [Geotools-devel] handling of geometry crs > > On 10-07-26 2:41 AM, Andrea Aime wrote: >> r

Re: [Geotools-devel] handling of geometry crs

2010-07-27 Thread Andrea Aime
Justin Deoliveira wrote: > The second way would be to modify the binding for AbstractFeatureType > and when it returns a geometry value to somehow set the srs there from > the feature type attribute type. Unfortunately simply modifying the user > data in place is probably not a good idea. It wo

Re: [Geotools-devel] handling of geometry crs

2010-07-27 Thread Andrea Aime
rini.angre...@csiro.au wrote: > Hi Justin, > > I like your second idea, and it is actually similar to the patch Jody whipped > up (http://jira.codehaus.org/secure/attachment/50280/GML2EncodingUtils.patch). > Since AbstractFeatureTypeBinding.getProperties() eventually calls > GML2EncodingUtils.ge

Re: [Geotools-devel] handling of geometry crs

2010-07-27 Thread Rini.Angreani
y, 27 July 2010 12:32 AM To: Andrea Aime Cc: Angreani, Rini (CESRE, Kensington); jody.garn...@gmail.com; geotools-devel@lists.sourceforge.net Subject: Re: [Geotools-devel] handling of geometry crs On 10-07-26 2:41 AM, Andrea Aime wrote: > rini.angre...@csiro.au wrote: >> Hi Jody, >>

Re: [Geotools-devel] handling of geometry crs

2010-07-26 Thread Justin Deoliveira
On 10-07-26 2:41 AM, Andrea Aime wrote: > rini.angre...@csiro.au wrote: >> Hi Jody, >> >> Ah, I see how the CRS gets updated in the geometry attribute now. >> However, for complex features, it doesn't even go through >> ReprojectingFeatureCollection, that's why the schema doesn't get >> recreated w

Re: [Geotools-devel] handling of geometry crs

2010-07-26 Thread Andrea Aime
rini.angre...@csiro.au wrote: > Hi Jody, > > Ah, I see how the CRS gets updated in the geometry attribute now. > However, for complex features, it doesn't even go through > ReprojectingFeatureCollection, that's why the schema doesn't get recreated > with the new CRS. > > My stack trace starts f

Re: [Geotools-devel] handling of geometry crs

2010-07-21 Thread Rini.Angreani
urceforge.net; jdeol...@opengeo.org Subject: Re: [Geotools-devel] handling of geometry crs No worries; I thought of a simpler fix that may work out for all concerned. In GML2EncodingUtils add .. else if( propety instanceof GeoemtryAttribute ){ Geometry geometry = (Geometr

Re: [Geotools-devel] handling of geometry crs

2010-07-21 Thread Jody Garnett
since I'm not sure how > to do it correctly. > Thanks for all your help so far. > > Cheers > Rini > > > > > > ____ > From: Jody Garnett [jody.garn...@gmail.com] > Sent: Wednesday, 21

Re: [Geotools-devel] handling of geometry crs

2010-07-21 Thread Rini.Angreani
reani, Rini (CESRE, Kensington) Cc: geotools-devel@lists.sourceforge.net Subject: Re: [Geotools-devel] handling of geometry crs Reni here is the code that creates the target feature type ... public ReprojectingFeatureCollection( SimpleFeatureCollection delegate, CoordinateRefere

Re: [Geotools-devel] handling of geometry crs

2010-07-20 Thread Jody Garnett
Reni here is the code that creates the target feature type ... public ReprojectingFeatureCollection( SimpleFeatureCollection delegate, CoordinateReferenceSystem source, CoordinateReferenceSystem target) { .. SimpleFeatureType schema = delegate.getSchema(

Re: [Geotools-devel] handling of geometry crs

2010-07-20 Thread Jody Garnett
> I think we need an IRC session; as we are starting to lose the forest for the > trees. Morning Reni: Let me know when you get in and we can go over this on IRC; I am afraid we have probably missed Justin for the day - you may need to make separate arrangements to talk to him. Justin we seem to

Re: [Geotools-devel] handling of geometry crs

2010-07-20 Thread Jody Garnett
On 20/07/2010, at 6:50 PM, wrote: > Hi Jody, > > I thought of creating my own binding as well, etc. After playing around with > this today, and came up with a dodgy patch on the binding side, I realised we > still need my original patch anyway (and more). I tried it on my test and it > does

Re: [Geotools-devel] handling of geometry crs

2010-07-20 Thread Rini.Angreani
(CESRE, Kensington) Cc: geotools-devel@lists.sourceforge.net Subject: Re: [Geotools-devel] handling of geometry crs Cool I think you have found the heart of the problem :-) We would like the encoder to pull out two values the geometry and the crs. Any way we can modify it to ignore GeometryAttri

Re: [Geotools-devel] handling of geometry crs

2010-07-19 Thread Jody Garnett
Cool I think you have found the heart of the problem :-) We would like the encoder to pull out two values the geometry and the crs. Any way we can modify it to ignore GeometryAttribute and provide our own binding for GeometryAttribute? Or pull out two values (the geometry and crs)? If you have

Re: [Geotools-devel] handling of geometry crs

2010-07-19 Thread Rini Angreani
Hi Jody, I think I understand the problem better now. In GML2EncodingUtils.AbstractFeatureType_getProperties(): if (property instanceof ComplexAttribute) { // do not unpack complex attributes as these may have their own bindings, which

Re: [Geotools-devel] handling of geometry crs

2010-07-18 Thread Rini Angreani
Thanks Jody. Sorry for the late reply, I only subscribe to the mailing list daily digest, so I didn't immediately get the post. There's already a GeometryPropertyTypeBinding, where it *may* have the geometry property instead of the geometry itself (like in AbstractGeometryTypeBinding), but I don't

Re: [Geotools-devel] handling of geometry crs

2010-07-16 Thread Jody Garnett
Have not managed to get Rini on the email list - she did add the following comment: > Thanks Jody. > I think, the way the encoder works, it parses the schema, and checks for each > property and attribute in the object. If it's there, it gets encoded. > As for geometries, AbstractGeometryTypeBind

[Geotools-devel] handling of geometry crs

2010-07-15 Thread Jody Garnett
Reni an outstanding patch for GEOT-2707 - and while the patch is clear I would like to take discussion to the devel list as it touches on a few cross module questions. Reni's patch correctly updates a CoordinateReferenceSystem object placed in a JTS Geometry userData. In the feature module we hav