Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-10-01 Thread Jody Garnett
Evening Victor: In the interest of getting this discussion to converge I have gathered the code examples here: - http://docs.codehaus.org/display/GEOTOOLS/Parameter+Interaction -- Jody Garnett -- Got visibility? Most

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-10-01 Thread Jody Garnett
I a considering combing the two ideas: a) Use GROUP to "indicate" the parameters in the same group work together b) Use INTERACTION to indicate how they interact, if not interaction is specified then the parameter considered the subject or topic of the group (and is thus "enabled", or describe

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-26 Thread Andrea Aime
On Wed, Sep 26, 2012 at 5:28 PM, Victor Olaya wrote: > > > > Could we have more than one relationship? How do we handle it? > > You mean one parameter related to many other ones? That could be > useful, but I guess it risks making it difficult to handle, since > probably additional information wa

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-26 Thread Jody Garnett
> Sounds great to me. I guess it's better to use an enum, instead of > string, but the syntax of those examples is perfect from my point of > view Well I want to see it run first, to make sure we are on the right track. > hmmm, I do not understand this one. The GROUP parameter I was > proposing

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-26 Thread Victor Olaya
> > Could we have more than one relationship? How do we handle it? You mean one parameter related to many other ones? That could be useful, but I guess it risks making it difficult to handle, since probably additional information was needed to fully describe that more complex relation. Can you put

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-26 Thread Andrea Aime
On Wed, Sep 26, 2012 at 3:22 PM, Victor Olaya wrote: > > RELATE > > > > How to describe the nature of the relationship? Using strings below? > Perhaps > > best handled with ENUM. > > > > Parameter SNAPPING = new Parameter("snapping", > > Boolean.class, "Enable Snapping", "True to enable snapping,

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-26 Thread Victor Olaya
> RELATE > > How to describe the nature of the relationship? Using strings below? Perhaps > best handled with ENUM. > > Parameter SNAPPING = new Parameter("snapping", > Boolean.class, "Enable Snapping", "True to enable snapping, requires use of > 'distance' parameter", true, 1, 1, new KVP( RELATED,

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-25 Thread Jody Garnett
Evening Victor (I started this message earlier in the day, and see that Justin has now covered some of the same topics). Great suggestions, however we will need to sort out a name. I often use code examples in order to create something that reads well to developers. I also ask that we hook the

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-24 Thread Victor Olaya
> > One advantage to hard-coding the relationship is that it gives better > type-checking and IDE support. Are there likely to be more than a few > different relationships that need to be expressed? I just can think of a band index from a raster layer (talking from what I have seen in GRASS proce

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-24 Thread Martin Davis
On Mon, Sep 24, 2012 at 8:24 AM, Victor Olaya wrote: > > >> > > I wonder if calling this a "relationship" rather than a dependency makes > > more sense. What sort of other information will the > relationship/dependency > > relay? Like for instance, in this could be good to create the > relationsh

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-24 Thread Victor Olaya
> Just to be clear the new stuff would be part of the Parameter class directly > right? and available to all process implementors, not just ones using the > annotation based stuff? Yes, I think that if it goes in the Parameter class it will be available to all processes. In fact, I think there are

Re: [Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-24 Thread Justin Deoliveira
I think this would be great to have, anything that increase the expressiveness available to the process author is a step up. Just to be clear the new stuff would be part of the Parameter class directly right? and available to all process implementors, not just ones using the annotation based stuff

[Geotools-devel] Modification to Parameter class for geoprocesses

2012-09-24 Thread Victor Olaya
Hi all I would like to propose two simple modifications to the Parameter class, to enhance the semantics of geoprocesses. They both can be implemented as new hints, and made available through the DescribeParameter annotation - A hint describing dependence of one parameter to another. The most typ