Revision 933

Comitted a patch as discussed with Martin in that email:

Hallo Stefan,

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

Darüberhinaus 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ür mich machen
diese Zeilen auch gar keinen Sinn... wieso muss man Min/Max vor dem
Schreiben die Werte zurücksetzen?
Hast Du eine Idee wozu das gemacht wird?
Selbst, wenn in der folgenden Schleife Min/Max neu berechnet würden,
wäre 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üssten!

Mein Vorschlag deshalb:
a) die Änderung so machen, wie Du vorschlägst
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über gestolpert, dass Fload.MIN_VALUE und
> > Double.MIN_VALUE nur positive! Werte zurücklifert. 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
> > Änderung in schmitzm vor. Aber da ich nicht ganz drinstecke überlasse
> > ich dir das zu ändern. Ist völlig unwichtig für GP, nur so aus prinzip.
> > 
> > Ist vielleicht auch nicht wichtig..
> > Gruß
> >  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;
> > 
> > 
> > 
> >