[Schmitzm-commits] r933 - trunk/src/gtmig/org/geotools/gce/arcgrid

[scm-commit@wald.intevation.org]

Author: alfonx
Date: 2010-07-08 11:37:25 +0200 (Thu, 08 Jul 2010)
New Revision: 933

Modified:
   trunk/src/gtmig/org/geotools/gce/arcgrid/ArcGridRaster.java
Log:
Comitted a patch as discussed with Martin in that email:

Hallo Stefan,

ich denke, das sollte man so machen... k?\195?\182nnte auch die Ursache sein,
dass die Visualisierung mit negativen Werten bisher glaube ich nicht so
richtig geklappt hat.

Dar?\195?\188berhinaus ist mir ist in der Klasse noch eine Sache aufgefallen:

public void writeRaster(Raster raster, double xl, double yl,
  ...
  // reset min and max
  minValue = Double.MIN_VALUE;
  maxValue = Double.MAX_VALUE;
  ...
}

Anschliessend wird Min/Max gar nicht weiter verwendet. F?\195?\188r mich machen
diese Zeilen auch gar keinen Sinn... wieso muss man Min/Max vor dem
Schreiben die Werte zur?\195?\188cksetzen?
Hast Du eine Idee wozu das gemacht wird?
Selbst, wenn in der folgenden Schleife Min/Max neu berechnet w?\195?\188rden,
w?\195?\164re die Initialisierung falsch, da minValue ja mit Double.MAX_VALUE
(bzw. POSITIVE_INFINITY) und maxValue mit Double.MIN_VALUE (bzw.
NEGATIVE_INFINITY) initialisiert werden m?\195?\188ssten!

Mein Vorschlag deshalb:
a) die ?\195?\132nderung so machen, wie Du vorschl?\195?\164gst
b) die beiden Zeilen oben einfach auskommentieren

Gruss

Martin


Am 07.07.2010 18:06, schrieb Stefan A. Tzeggai:
> > Hi martin
> > 
> > Ich bin eben selber dr?\195?\188ber gestolpert, dass Fload.MIN_VALUE und
> > Double.MIN_VALUE nur positive! Werte zur?\195?\188cklifert. Somit kann ein
> > maxValue nicht kleiner 0 werden, wenn man das als Startwert benutzt und
> > mit Math.max vergleicht.
> > 
> > Ich hab dann mal nach Verwendung gesucht, und schlage dir folgende
> > ?\195?\132nderung in schmitzm vor. Aber da ich nicht ganz drinstecke ?\195?\188berlasse
> > ich dir das zu ?\195?\164ndern. Ist v?\195?\182llig unwichtig f?\195?\188r GP, nur so aus prinzip.
> > 
> > Ist vielleicht auch nicht wichtig..
> > Gru?\195?\159
> >  und CU Samstag!
> > 
> > 
> > Index: ArcGridRaster.java
> > ===================================================================
> > --- ArcGridRaster.java	(revision 930)
> > +++ ArcGridRaster.java	(working copy)
> > @@ -85,10 +85,10 @@
> >      private URL srcURL;
> > 
> >      /** max value found in the file */
> > -    protected double maxValue = Float.MIN_VALUE;
> > +    protected double maxValue = Double.NEGATIVE_INFINITY;
> > 
> >      /** min value found in the file */
> > -    protected double minValue = Float.MAX_VALUE;
> > +    protected double minValue = Double.POSITIVE_INFINITY;
> > 
> >      protected double xllCorner = Double.NaN;
> > 
> > 
> > 
> > 


Modified: trunk/src/gtmig/org/geotools/gce/arcgrid/ArcGridRaster.java
===================================================================
--- trunk/src/gtmig/org/geotools/gce/arcgrid/ArcGridRaster.java	2010-07-05 21:48:14 UTC (rev 932)
+++ trunk/src/gtmig/org/geotools/gce/arcgrid/ArcGridRaster.java	2010-07-08 09:37:25 UTC (rev 933)
@@ -85,10 +85,10 @@
     private URL srcURL;
 
     /** max value found in the file */
-    protected double maxValue = Float.MIN_VALUE;
+    protected double maxValue = Double.NEGATIVE_INFINITY;
 
     /** min value found in the file */
-    protected double minValue = Float.MAX_VALUE;
+    protected double minValue = Double.POSITIVE_INFINITY;
 
     protected double xllCorner = Double.NaN;
 
@@ -557,9 +557,9 @@
         spaces(out, 2);
         out.println(noData);
 
-        // reset min and max
-        minValue = Double.MIN_VALUE;
-        maxValue = Double.MAX_VALUE;
+//        // reset min and max
+//        minValue = Double.MIN_VALUE;
+//        maxValue = Double.MAX_VALUE;
 
         // a buffer to flush each line to
         // this technique makes things a bit quicker because buffer.append()