From: Jody G. <jod...@gm...> - 2012-07-17 02:01:41
|
Another doc page for your proposal: - http://docs.geotools.org/latest/userguide/library/api/envelope.html -- Jody Garnett On Tuesday, 17 July 2012 at 11:54 AM, Jody Garnett wrote: > So I kind of see where you are going with this. It gets a bit strange around the ReferencedEnvelope / ReferencedEnvelope3D level. > > -1 > > The FeatureCollections.getBounds() contract cannot be respected for your ReferenceEnvelope or ReferenceEnvelope3D. I know you only want to use this as part of a query, but I don't want our envelope story to get even more strange if we can avoid it. > > I don't want to impose a ReferenceEnvelope interface as that defeats the purpose of making ReferenceEnvelope an easy to use middle ground between JTS Envelope and the BoundingBox interface. If we had to go this way we would end up going with BoundingBox / Envelope interface and modifying them to be more method compatible with JTS Envelope. > > Can you consider switching around your class hierarchy? > > 1) BBoxEnvelope extension of JTS Envelope > - Write this so it can work as a 2D or 3D envelope (depending on how the constructor is called) > - the existing BoundingBox / Envelope interfaces should be sufficient? > 2) Make ReferencedEnvelope extend BBoxEnvelope; and allow it to cover the 2D or 3D use-case > > Finally there is no need to make FilterFactory more crazy, treat your ReferencedEnvelope as a prameter object: > > BBOX3D bbox( Expression geometry, ReferencedEnvelope bbox, MatchAction matchAction); > > Waiting your reply, I am afraid we cannot make a ReferencedEnvelope interface (as an implementation class lots of downstream code makes direct reference to this one). > -- > Jody Garnett > > > Forwarded message: > > > From: Niels Charlier <ni...@sc... (mailto:ni...@sc...)> > > To: geo...@li... (mailto:geo...@li...) > > Date: Tuesday, 17 July 2012 6:09:35 AM > > Subject: [Geotools-devel] *proposal*: Support for three-dimensional envelopes and bounding box filters > > > > Hi Everyone, > > > > Please have a look at my proposal: > > http://docs.codehaus.org/display/GEOTOOLS/Support+for+three-dimensional+envelopes+and+bounding+box+filters > > > > Please review. > > > > Kind Regards > > Niels Charlier > > > > ------------------------------------------------------------------------------ > > Live Security Virtual Conference > > Exclusive live event will cover all the ways today's security and > > threat landscape has changed and how IT managers can respond. Discussions > > will include endpoint security, mobile security and the latest in malware > > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > > _______________________________________________ > > GeoTools-Devel mailing list > > Geo...@li... (mailto:Geo...@li...) > > https://lists.sourceforge.net/lists/listinfo/geotools-devel > > > > > > > > > > |