Sorry about that.  That's from testing and should not have made it in the
final product.  I'll be sending another PR shortly.


On Thu, Sep 19, 2013 at 10:56 AM, Rob Emanuele <notifications@github.com>wrote:

> In src/main/scala/geotrellis/raster/op/zonal/summary/Mean.scala:
>
> > +  def createTileResults(trd:TiledRasterData, re:RasterExtent) = {
> > +    trd.getTiles(re)
> > +       .map { r => (r.rasterExtent, meanTileRaster(r))}
> > +       .toMap
> > +  }
> > +
> > +  def meanRaster (r:Raster):Double = {
> > +    meanTileRaster(r).mean
> > +  }
> > +
> > +  def meanTileRaster (r:Raster):DoubleMean = {
> > +
> > +    var sum = 0.0
> > +    var count = 0L
> > +    r.foreachDouble( (x) => if (!java.lang.Double.isNaN(x)) { sum = sum + x; count = count + 1 })
> > +    println("Sum: " + sum + " Count: " + count + " Result: " + sum/count)
>
> Having println()'s in operations have caused some slowdown in execution
> time. Can these be removed?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/geotrellis/geotrellis/pull/564/files#r6464217>
> .
>