Re: [Geotools-devel] Graduating gt-transform to supported status

2013-07-06 Thread Andrea Aime
On Fri, Jun 14, 2013 at 2:22 PM, Andrea Aime wrote: > Hi, > I'm writing you to start the process to graduate the gt-transform module > to supported status. > Hi all, here is the pull request to graduate the module: https://github.com/geotools/geotools/pull/224 In the end I picked up Jody's sugge

Re: [Geotools-devel] Graduating gt-transform to supported status

2013-06-16 Thread Ben Caradoc-Davies
+1 for putting it in library. This looks almost like it could go into main, but keeping it in a separate module is great if you can. -0 for factories. I think this is a component that could in the future be used to build a transforming factory; I do not think that you should feel obliged to wri

Re: [Geotools-devel] Graduating gt-transform to supported status

2013-06-16 Thread Jody Garnett
Move it later if/when that happens. Oh and +1 it is a great bit of functionality. -- Jody Garnett On Saturday, 15 June 2013 at 4:41 PM, Andrea Aime wrote: > On Sat, Jun 15, 2013 at 8:21 AM, Jody Garnett (mailto:jody.garn...@gmail.com)> wrote: > > > Ah, the final location is a bit of a myst

Re: [Geotools-devel] Graduating gt-transform to supported status

2013-06-14 Thread Andrea Aime
On Sat, Jun 15, 2013 at 8:21 AM, Jody Garnett wrote: > Ah, the final location is a bit of a mystery... this thing is not a plugin > in the strict sense, there is no factory and no store, you are supposed to > call TransformFactory.transform(source, targetName, > listOfAttributeDefinitions) in orde

Re: [Geotools-devel] Graduating gt-transform to supported status

2013-06-14 Thread Jody Garnett
> All of the above is achieved by associating the new set of attributes each > with a OGC Expression, like in the test cases, see the three methods setting > up some transformations: > > https://github.com/geotools/geotools/blob/master/modules/unsupported/transform/src/test/java/org/geotools/dat

Re: [Geotools-devel] Graduating gt-transform to supported status

2013-06-14 Thread Justin Deoliveira
On Fri, Jun 14, 2013 at 6:22 AM, Andrea Aime wrote: > Hi, > I'm writing you to start the process to graduate the gt-transform module > to supported status. > > For those that don't know, the gt-transform module provides a way to > create wrappers around SimpleFeatureSource/SimpleFeatureStore that

[Geotools-devel] Graduating gt-transform to supported status

2013-06-14 Thread Andrea Aime
Hi, I'm writing you to start the process to graduate the gt-transform module to supported status. For those that don't know, the gt-transform module provides a way to create wrappers around SimpleFeatureSource/SimpleFeatureStore that do alter the attributes exposed via: * attribute selection * att