Fix - do not draw negative bins in 2d/3d scatter plots

Author: linev

scripts/JSRoot3DPainter.js

@@ -3297,13 +3297,14 @@
       if ((i2<=i1) || (j2<=j1) || (k2<=k1)) return true;
 
       // scale down factor if too large values
-      var coef = (this.gmaxbin > 1000) ? 1000/this.gmaxbin : 1, numpixels = 0;
+      var coef = (this.gmaxbin > 1000) ? 1000/this.gmaxbin : 1,
+          numpixels = 0, content_lmt = Math.max(0, this.gminbin);
 
       for (i = i1; i < i2; ++i) {
          for (j = j1; j < j2; ++j) {
             for (k = k1; k < k2; ++k) {
                bin_content = histo.getBinContent(i+1, j+1, k+1);
-               if (bin_content <= this.gminbin) continue;
+               if (bin_content <= content_lmt) continue;
                numpixels += Math.round(bin_content*coef);
             }
          }
@@ -3319,7 +3320,7 @@
          for (j = j1; j < j2; ++j) {
             for (k = k1; k < k2; ++k) {
                bin_content = histo.getBinContent(i+1, j+1, k+1);
-               if (bin_content <= this.gminbin) continue;
+               if (bin_content <= content_lmt) continue;
                var num = Math.round(bin_content*coef);
 
                for (var n=0;n<num;++n) {

scripts/JSRootPainter.more.js

@@ -4338,7 +4338,7 @@
       for (i = handle.i1; i < handle.i2; ++i) {
          for (j = handle.j1; j < handle.j2; ++j) {
             binz = histo.getBinContent(i + 1, j + 1);
-            if ((binz == 0) || (binz < this.minbin)) continue;
+            if ((binz <= 0) || (binz < this.minbin)) continue;
 
             cw = handle.grx[i+1] - handle.grx[i];
             ch = handle.gry[j] - handle.gry[j+1];