Ciao Frank, please, find my comments inline below... Regards, Simone Giannecchini == GeoServer training in Milan, 6th & 7th June 2013! Visit http://geoserver.geo-solutions.it for more information. ==
Ing. Simone Giannecchini @simogeo Founder/Director GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 333 8128928 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- On Wed, May 8, 2013 at 9:51 PM, Frank Warmerdam <warmer...@pobox.com> wrote: > Andrea, > > I'm not currently a user of the grid coverage reader, but I might at some > point. Skimming the proposal I was wondering: > > 1) Why you offer the getMetadataNames(coverage,...) and > getMetadataValues(coverage,...) on GridCoverageReader instead of handling > them on GridCoverage. > > 2) Similarly, why is the new GridCoverage2DReader interface on > GridCoverageReader instead of offering that interface on the GridCoverage. > > I'm guessing it is assumed the reader can provide some overall information > cheaply while it is assumed that the read() operation returning a > GridCoverage is expensive. But it *feels* like an awkward arrangement. > The original infrastructure for IO follows loosely the old GridCoverageExchange interfaces so that's why the split between reader and coverage. Moreover the way it was originally implemented follows close the JAI paradigm of working which is: - ImageRead to read - ImageWrite to write - RenderedImage to convey real data with tiling and deferred loading This is not *that* similar to how gdal works although there are similarities as you noticed below. > Not surprisingly your proposal is loosely similar to the handling of > "subdatasets" in GDAL though in GDAL there is no distinct concept of a > reader. It is just that a dataset can have subdatasets and that for > container formats like HDF the base dataset might not have any actual grid > coverage associated with it (ie. zero bands). > > Best regards, > Frank > > > > On Wed, May 8, 2013 at 8:34 AM, Andrea Aime <andrea.a...@geo-solutions.it> > wrote: >> >> Hi all, >> here is our overhauled proposal for multicoverage grid readers: >> >> http://docs.codehaus.org/display/GEOTOOLS/Efficient+support+for+multiple+coverages+in+GridCoverageReader >> >> This proposal allows to break the 1-1 relationship between reader and >> coverage, and allows proper support for multicoverage sources, such as >> spatial databases, ArcSDE, NetCDF (often they contain more than one coverage >> per file). >> >> Feedback, questions, and votes welcomed :-) >> >> Cheers >> Andrea >> >> -- >> == >> GeoServer training in Milan, 6th & 7th June 2013! Visit >> http://geoserver.geo-solutions.it for more information. >> == >> >> Ing. Andrea Aime >> @geowolf >> Technical Lead >> >> GeoSolutions S.A.S. >> Via Poggio alle Viti 1187 >> 55054 Massarosa (LU) >> Italy >> phone: +39 0584 962313 >> fax: +39 0584 1660272 >> mob: +39 339 8844549 >> >> http://www.geo-solutions.it >> http://twitter.com/geosolutions_it >> >> ------------------------------------------------------- >> >> >> ------------------------------------------------------------------------------ >> Learn Graph Databases - Download FREE O'Reilly Book >> "Graph Databases" is the definitive new guide to graph databases and >> their applications. This 200-page book is written by three acclaimed >> leaders in the field. The early access version is available now. >> Download your free book today! http://p.sf.net/sfu/neotech_d2d_may >> _______________________________________________ >> GeoTools-Devel mailing list >> GeoTools-Devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/geotools-devel >> > > > > -- > ---------------------------------------+-------------------------------------- > I set the clouds in motion - turn up | Frank Warmerdam, > warmer...@pobox.com > light and sound - activate the windows | http://pobox.com/~warmerdam > and watch the world go round - Rush | Geospatial Software Developer > > ------------------------------------------------------------------------------ > Learn Graph Databases - Download FREE O'Reilly Book > "Graph Databases" is the definitive new guide to graph databases and > their applications. This 200-page book is written by three acclaimed > leaders in the field. The early access version is available now. > Download your free book today! http://p.sf.net/sfu/neotech_d2d_may > _______________________________________________ > GeoTools-Devel mailing list > GeoTools-Devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel > ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. This 200-page book is written by three acclaimed leaders in the field. The early access version is available now. Download your free book today! http://p.sf.net/sfu/neotech_d2d_may _______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
