Hi Jody,
Thanks for the feedbacks.
On Wed, Mar 20, 2013 at 11:27 PM, Jody Garnett <[email protected]>wrote:
> First up I love the idea (Andrea also mentioned it on Monday's
> meeting). I am also keen to hear what Yancy Matherne the NetCDF crew (i.e.
> Yancy) think in terms community feedback.
>
> I know it was there before, but reader.listSubnames() mixes programming
> styles with respect to reader.getCoverageCount(). Is there scope to
> deprecate this method and introduce reader.getCoverageNames() to make the
> API consistent.
>
Agreed... changing it right now.
>
> A couple newbie questions:
> -previously were the values of listSubnames() were directly with
> reader.read( name )
>
Not sure to understand that. Andrea told me that he will provide you more
feedbacks on this.
> - Is reader.listSubnames().length the same thing as getCoverageCount()?
>
Right.
Cheers,
Daniele
>
> --
> Jody Garnett
>
> On Wednesday, 20 March 2013 at 9:41 PM, Daniele Romagnoli wrote:
>
> Hi list,
> The current implementation status of the GridCoveargeReader normally
> associates a single reader with a single coverage (e.g., TIFF, GTOPO30, and
> so on).
> It would be great to break the 1-1 association and allow
> GridCoverageReader to expose multiple covearges in an efficient way, in
> order to properly deal with sources storing multiple coverages with
> different structures and features (e.g., not mosaics, which can deal with
> multiple coverages having some degree of uniformity).
>
> The current *GridCoverageReader* interface already allows to expose
> multiple coverages, but in an inefficient and thread unsafe way, as it is
> intended to (quoting from the current JAVADOC)* "read GridCoverages from
> the input stream in a sequential order*".
> No form of random access support is available right now.
> Therefore, we propose to change that API in order to deprecate methods
> related to the current sequential access paradigm like "skip",
> "hasMoreGridCoverages", "getCurrentSubname" and add a few of them which
> allows "random acces" by name.
> Moreover, we propose also to add a new *GridCoverage2DReader* interface
> collecting common methods from *AbstractGridCoverage2DReader* which is
> the base class implementation.
>
> As the API changes proposal requires, I'm pasting here the JIRA link as
> well as the proposal page which contains more detailed information on the
> topic, methods change proposal and interfaces related to that.
>
> https://jira.codehaus.org/browse/GEOT-4424
>
> http://docs.codehaus.org/display/GEOTOOLS/Efficient+support+for+multiple+coverages+in+GridCoverageReader
>
> Cheers,
> Daniele
>
> --
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
> ==
>
> Ing. Daniele Romagnoli
> Senior Software Engineer
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_d2d_mar
> _______________________________________________
> GeoTools-Devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
>
--
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==
Ing. Daniele Romagnoli
Senior Software Engineer
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
