On Sun, Feb 24, 2008 at 1:33 PM, Martin Desruisseaux
<[EMAIL PROTECTED]> wrote:
> I'm 0 (neutral) on the Node and Visitor classes - I don't really know how they
>  fit in the big picture because of my lack of knowledge of styling and 
> streaming
>  renderer.
>
>  However I ask to remove from the proposal the new Category hierarchy. I 
> realize
>  that the existing Category class may need to be extented in order to meet
>  RasterSymbolizer needs, but I'm in disagreement with the TransformCategory 
> split
>  proposed there.
>

What do you mean by "remove"? The code to actually do the raster
symbolization heavily relies on this code it cannot be simply
"removed".
What do you suggest?


>  The proposed Category hierarchy duplicates org.geotools.coverage.Category
>  introducing confusion if they appear in public API (and they will since they
>  would be included in the javadoc).

>  I'm in disagreement with Simone's view about
>  "sampleToGeophysics". It is a packaged form of "scale", "offset" and "nodata"
>  attributes which anyone can found in NetCDF, HDF and other file formats and 
> is
>  even part of OGC specification. It is closely tied to Unit of Measurement, 
> which
>  is indisociable from Coverage data (this is also OGC specification). It is
>  conceptually related to "gridToCRS" transform, which is also indisociable 
> from
>  any Coverage.

1>as I pointed out in the other email, we are not understanding each
other here, I hope that email will bring some clarity.
2>This does not entirely fits this proposal, since it is not
discussing about refactoring coverage



>
>  The "Raster symbolizer" proposal actually mixes both a raster symbolizer and 
> a
>  refactoring of Category - even if it is in a separated package. I'm against 
> the
>  later. I'm not against a refactoring of Category, but I disagree with the one
>  which is proposed on that page and I'm against the fact that this refactoring
>  would be done in a separated package.

I definitely see your point, this duplication of names can be confusing.
I am going to think about a way to hide the classification categories
for the moment in order to bring up the discussion again in the
coverge context.

Simone.

>
>         Martin
>
>
>
>  -------------------------------------------------------------------------
>  This SF.net email is sponsored by: Microsoft
>  Defy all challenges. Microsoft(R) Visual Studio 2008.
>  http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
>  _______________________________________________
>  Geotools-devel mailing list
>  Geotools-devel@lists.sourceforge.net
>  https://lists.sourceforge.net/lists/listinfo/geotools-devel
>



-- 
-------------------------------------------------------
Eng. Simone Giannecchini
President /CEO GeoSolutions S.A.S.
Via Carignoni 51
55041  Camaiore (LU)
Italy

phone: +39 0584983027
fax:      +39 0584983027
mob:    +39 333 8128928


http://www.geo-solutions.it

-------------------------------------------------------

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to