I am happy if you get additional deprecations in before the 13.0 release
(checking for deprecated classes is often one of the last things we do
before release). I am not close enough to the coverage model to comment on
removing view type. It seems a bit odd to reduce the question to just one
of rendering via SLD (I could see analysis requiring the ability to
discover what content is stored in the raster).
I guess I would like to hear from Andrea Antonello (or similar) who uses
raster data in a scientific context.
Jody
--
Jody Garnett
On 5 February 2015 at 05:38, Daniele Romagnoli <
daniele.romagn...@geo-solutions.it> wrote:
> Hi List,
> I would like to simplify the GridCoverage machinery by deprecating
> concepts of Coverage's ViewType and related classes such as ViewsManager,
> SampleToGeophysics math transformation,....
>
> Most parts of the code related to these concepts are indeed only useful
> when you deal with a coverage which contains packed data whilst in the vast
> majority of cases we always deal with unpacked datasets. Moreover,
> geophysics data can be properly rendered through SLD (by means of colormaps
> and rendering transformations) as preferred by the data user.
> Beside of that, these classes are sometime subject of unneeded processing
> steps (as the one performed by the ViewsManager to create the different
> views Native/Packed/Geophysic/Photographic of a coverage) as well as
> unexpected behaviours in some cases. As an instance, the inner scan of all
> the data values performed to determine the categories range to find the
> inverse transformation (and inverse range) when some parameters are
> missing. This resulted in performance loss on big datasets. Finally, there
> is some other auto-magic logic happening there which led to hard debugging
> times in the past. It tooks a while to figure out what was going wrong with
> some components of the coverage (sample and color model) complaining
> against some underlying view's assumptions.
>
> Therefore, I would like to deprecate their usage in the code (after the
> freeze) on 14.x (removing them on 15.x) from the GridCoverage
> loading/processing machinery by making sure that things still continue
> working as before.
> This will result, as an instance, in stop asking for geophysic view of a
> dataset (with respect to its native one, or the packed one) and just get
> the data as is. The reader will make sure to deal with corner cases if
> there are. Parts of code which was eventually dealing with different views,
> will be updated accordingly.
>
> Does anyone has any objection on this? (I will obviously create a JIRA and
> wait for the freeze)
>
> Please, let me know.
> Cheers,
> Daniele
>
>
>
>
> ==
> GeoServer Professional Services from the experts! Visit
> http://goo.gl/NWWaa2 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
>
> -------------------------------------------------------
>
> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>
> Le informazioni contenute in questo messaggio di posta elettronica e/o
> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
> darcene notizia via e-mail e di procedere alla distruzione del messaggio
> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
> principi dettati dal D.Lgs. 196/2003.
>
>
>
> The information in this message and/or attachments, is intended solely for
> the attention and use of the named addressee(s) and may be confidential or
> proprietary in nature or covered by the provisions of privacy act
> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
> Code).Any use not in accord with its purpose, any disclosure, reproduction,
> copying, distribution, or either dissemination, either whole or partial, is
> strictly forbidden except previous formal approval of the named
> addressee(s). If you are not the intended recipient, please contact
> immediately the sender by telephone, fax or e-mail and delete the
> information in this message that has been received in error. The sender
> does not give any warranty or accept liability as the content, accuracy or
> completeness of sent messages and accepts no responsibility for changes
> made after they were sent or for other risks which arise as a result of
> e-mail transmission, viruses, etc.
>
>
>
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming. The Go Parallel Website,
> sponsored by Intel and developed in partnership with Slashdot Media, is
> your
> hub for all things parallel software development, from weekly thought
> leadership blogs to news, videos, case studies, tutorials and more. Take a
> look and join the conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
