Re: [Geotools-devel] WMSExtent

2010-05-21 Thread Jody Garnett
It should already be in the snapshot from last night. Jody On 21/05/2010, at 6:26 PM, Heimo Laukkanen wrote: > On Fri, May 21, 2010 at 11:17 AM, Jody Garnett wrote: > >> Returning something more meaningful (such as a "range") can be a topic for >> another day; and indeed I hope you can contribu

Re: [Geotools-devel] WMSExtent

2010-05-21 Thread Heimo Laukkanen
On Fri, May 21, 2010 at 11:17 AM, Jody Garnett wrote: > Returning something more meaningful (such as a "range") can be a topic for > another day; and indeed I hope you can contribute some useful methods to the > Extent class as you need to? I'll keep you posted, when we do something worthwhile o

Re: [Geotools-devel] WMSExtent

2010-05-21 Thread Jody Garnett
> thanks a bunch for pointing me to the right direction and helping me > to understand what it is that makes Geotools' parsing work. Sorry it is going to be a slog to work through. As I mentioned you will have company next week. > What you wrote above is correct, but you quite lost me with the r

Re: [Geotools-devel] WMSExtent

2010-05-21 Thread Heimo Laukkanen
On Thu, May 20, 2010 at 5:17 PM, Jody Garnett wrote: > Okay let me try again ... > gt-wms > == > org.geotools.data.wms.xml.Extent > -this is a class it does not implement any interfaces > - the isEmpty fix is applied here :-) > - this is the end of the line no modules depend on wms so nothing

Re: [Geotools-devel] WMSExtent

2010-05-20 Thread Jody Garnett
On 20/05/2010, at 11:47 PM, Heimo Laukkanen wrote: > On Thu, May 20, 2010 at 4:07 PM, Jody Garnett wrote: >> I am not sure I get it what are we trying to do with this change? > > There isn't a way to get the data ( extent ) from the dimension. Okay we have two many things called extent - we bet

Re: [Geotools-devel] WMSExtent

2010-05-20 Thread Jody Garnett
Okay let me try again ... gt-wms == org.geotools.data.wms.xml.Extent -this is a class it does not implement any interfaces - the isEmpty fix is applied here :-) - this is the end of the line no modules depend on wms so nothing can extent org.geotools.data.wms.xml.Extent - this class could im

Re: [Geotools-devel] WMSExtent

2010-05-20 Thread Heimo Laukkanen
On Thu, May 20, 2010 at 4:07 PM, Jody Garnett wrote: > I am not sure I get it what are we trying to do with this change? Well at the moment, when you parse getCapabilities responses and have layers with dimensions and dimension extents, for example: hydro.runoffmaxaverage Average maximum runoff

[Geotools-devel] WMSExtent

2010-05-20 Thread Jody Garnett
I am not sure I get it what are we trying to do with this change? On 20/05/2010, at 9:40 PM, Heimo Laukkanen wrote: > As a sidenote to the WMS module, could someone make following fix to > the Extent class, unless it breaks something else. > > public class Extent extends org.geotools.data.wms.xm