Merge branch 'master' of http://github.com/danvk/dygraphs
authorNeal Nelson <neal@makalumedia.com>
Thu, 10 Feb 2011 10:22:22 +0000 (11:22 +0100)
committerNeal Nelson <neal@makalumedia.com>
Thu, 10 Feb 2011 10:22:22 +0000 (11:22 +0100)
Conflicts:
docs/index.html
dygraph.js

1  2 
docs/index.html
dygraph.js

diff --combined docs/index.html
@@@ -70,6 -70,7 +70,7 @@@
          <li><a href="tests/two-series.html">Multiple Series</a></li>
          <li><a href="tests/highlighted-region.html">Custom Underlay / background</a></li>
          <li><a href="tests/zoom.html">Tests for zoom operations</a></li>
+         <li><a href="tests/logscale.html">Log scale tests</a></li>
        </ul>
      </div>
  
@@@ -447,85 -448,6 +448,85 @@@ new Dygraph(el, data, 
        <li>Where the error bars do not overlap, we can say with 95% confidence that the series differ. There is a better than 95% chance that Ichiro was a better hitter than his team as a whole in 2004, the year he won the batting title.</li>
      </ul>
  
 +    <h2 id="zoom">Determining Zoom</h2>
 +    
 +    <p>
 +      It is possible to detect whether a chart has been zoomed in either axis by the use of the <code>isZoomed</code> function.
 +      If called with no argument, it will report whether <em>either</em> axis has been zoomed.
 +      Alternatively it can be called with an argument of either <code>'x'</code> or <code>'y'</code> and it will report the status of just that axis.
 +    </p>
 +
 +    <p>Here's a simple example using <code>drawCallback</code> to display the various zoom states whenever the chart is zoomed:</p>
 +
 +    <div style="width:600px; text-align:center; font-weight: bold; font-size: 125%;">OUTPUT</div>
 +    <div style="width: 750px">
 +      <div style="float: right">
 +          <p>Zoomed: <span id="zoomed">False</span><p/>
 +          <p>Zoomed X: <span id="zoomedX">False</span><p/>
 +          <p>Zoomed Y: <span id="zoomedY">False</span><p/>
 +      </div>
 +      <div class="codeoutput" style="float:left;">
 +        <div id="zoomdiv"></div>
 +        <script type="text/javascript">
 +          new Dygraph(
 +
 +            // containing div
 +            document.getElementById("zoomdiv"),
 +
 +            // CSV or path to a CSV file.
 +            "Date,Value\n" +
 +            "2011-01-07,75\n" +
 +            "2011-01-08,70\n" +
 +            "2011-01-09,90\n" +
 +            "2011-01-10,30\n" +
 +            "2011-01-11,40\n" +
 +            "2011-01-12,60\n" +
 +            "2011-01-13,70\n" +
 +            "2011-01-14,40\n",
 +            {
 +              drawCallback: function(me, initial) {
 +                document.getElementById("zoomed").innerHTML = "" + me.isZoomed();
 +                document.getElementById("zoomedX").innerHTML = "" + me.isZoomed("x");
 +                document.getElementById("zoomedY").innerHTML = "" + me.isZoomed("y");
 +              }
 +            }
 +          );
 +        </script>
 +      </div>
 +    </div>
 +
 +    <p>
 +      <div style="clear:both; width:600px; text-align:center; font-weight: bold; font-size: 125%;">HTML</div>
 +
 +<pre>
 +  new Dygraph(
 +
 +    // containing div
 +    document.getElementById(&quot;zoomdiv&quot;),
 +
 +    // CSV or path to a CSV file.
 +    &quot;Date,Temperature\n&quot; +
 +    &quot;2011-01-07,75\n&quot; +
 +    &quot;2011-01-08,70\n&quot; +
 +    &quot;2011-01-09,90\n&quot; +
 +    &quot;2011-01-10,30\n&quot; +
 +    &quot;2011-01-11,40\n&quot; +
 +    &quot;2011-01-12,60\n&quot; +
 +    &quot;2011-01-13,70\n&quot; +
 +    &quot;2011-01-14,40\n&quot;,
 +    {
 +      drawCallback: function(me, initial) {
 +        document.getElementById(&quot;zoomed&quot;).innerHTML = &quot;&quot; + me.isZoomed();
 +        document.getElementById(&quot;zoomedX&quot;).innerHTML = &quot;&quot; + me.isZoomed(&quot;x&quot;);
 +        document.getElementById(&quot;zoomedY&quot;).innerHTML = &quot;&quot; + me.isZoomed(&quot;y&quot;);
 +      }
 +    }
 +  );
 +</pre>
 +    </p>
 +
 +    <p>The <a href="tests/zoom.html">Tests for zoom operations</a> show a full example of this in action.</p>
 +
      <h2 id="stock">One last demo</h2>
  
      <p>This chart shows monthly closes of the Dow Jones Industrial Average, both in nominal and real (i.e. adjusted for inflation) dollars. The shaded areas show its monthly high and low. CPI values with a base from 1982-84 are used to adjust for inflation.</p>
@@@ -1026,13 -948,18 +1027,28 @@@ perl -ne 'BEGIN{print "Month,Nominal,Re
          </tr>
  
          <tr>
++<<<<<<< HEAD
 +          <td><strong>noZoomFlagChange</strong></td>
 +          <td><code></code></td>
 +          <td><code></code></td>
 +          <td>
 +            When this flag is passed along with either the <code>dateWindow</code> or <code>valueRange</code> options, the zoom flags are not changed to reflect a zoomed state.
 +            This is primarily useful for when the display area of a chart is changed programmatically and also where manual zooming is allowed and use is made of the <code>isZoomed</code> method to determine this.
 +            <div class="tests">Tests: <a href="tests/no-zoom-change.html">no-zoom-change</a></div>
++=======
+           <td><strong>logscale</strong></td>
+           <td><code>boolean</code></td>
+           <td><code>false</code></td>
+           <td>
+             When set for a y-axis, the graph shows that axis in y-scale. Any values less than or equal
+             to zero are not displayed.</p>
+           Not compatible with showZero, and ignores connectSeparatedPoints. Also, showing log scale
+           with valueRanges that are less than zero will result in an unviewable graph.<br/>
+             <div class="tests">Tests: <a href="tests/logscale.html">logscale</a>,
+             <a href="tests/stock.html"> stock</div>
++>>>>>>> a6a505d1759f087a1e29ad84fc8510aa4cbc4f6e
            </td>
          </tr>
  
diff --combined dygraph.js
@@@ -24,7 -24,6 +24,6 @@@
  
   If the 'errorBars' option is set in the constructor, the input should be of
   the form
     Date,SeriesA,SeriesB,...
     YYYYMMDD,A1,sigmaA1,B1,sigmaB1,...
     YYYYMMDD,A2,sigmaA2,B2,sigmaB2,...
@@@ -79,6 -78,11 +78,11 @@@ Dygraph.DEFAULT_WIDTH = 480
  Dygraph.DEFAULT_HEIGHT = 320;
  Dygraph.AXIS_LINE_WIDTH = 0.3;
  
+ Dygraph.LOG_SCALE = 10;
+ Dygraph.LN_TEN = Math.log(Dygraph.LOG_SCALE);
+ Dygraph.log10 = function(x) {
+   return Math.log(x) / Dygraph.LN_TEN;
+ }
  
  // Default attribute values.
  Dygraph.DEFAULT_ATTRS = {
  
    delimiter: ',',
  
-   logScale: false,
    sigma: 2.0,
    errorBars: false,
    fractions: false,
@@@ -195,10 -198,6 +198,10 @@@ Dygraph.prototype.__init__ = function(d
    this.is_initial_draw_ = true;
    this.annotations_ = [];
  
 +  // Zoomed indicators - These indicate when the graph has been zoomed and on what axis.
 +  this.zoomed_x_ = false;
 +  this.zoomed_y_ = false;
 +
    // Clear the div. This ensure that, if multiple dygraphs are passed the same
    // div, then only one will be drawn.
    div.innerHTML = "";
    this.start_();
  };
  
- // axis is an optional parameter. Can be set to 'x' or 'y'.
++// Axis is an optional parameter. Can be set to 'x' or 'y'.
 +Dygraph.prototype.isZoomed = function(axis) {
 +  if (axis == null) return this.zoomed_x_ || this.zoomed_y_;
 +  if (axis == 'x') return this.zoomed_x_;
 +  if (axis == 'y') return this.zoomed_y_;
 +  throw "axis parameter to Dygraph.isZoomed must be missing, 'x' or 'y'.";
 +};
 +
+ Dygraph.prototype.toString = function() {
+   var maindiv = this.maindiv_;
+   var id = (maindiv && maindiv.id) ? maindiv.id : maindiv
+   return "[Dygraph " + id + "]";
+ }
  Dygraph.prototype.attr_ = function(name, seriesName) {
    if (seriesName &&
        typeof(this.user_attrs_[seriesName]) != 'undefined' &&
@@@ -368,46 -365,154 +377,154 @@@ Dygraph.prototype.yAxisRanges = functio
   * If specified, do this conversion for the coordinate system of a particular
   * axis. Uses the first axis by default.
   * Returns a two-element array: [X, Y]
+  *
+  * Note: use toDomXCoord instead of toDomCoords(x, null) and use toDomYCoord
+  * instead of toDomCoords(null, y, axis).
   */
  Dygraph.prototype.toDomCoords = function(x, y, axis) {
-   var ret = [null, null];
+   return [ this.toDomXCoord(x), this.toDomYCoord(y, axis) ];
+ };
+ /**
+  * Convert from data x coordinates to canvas/div X coordinate.
+  * If specified, do this conversion for the coordinate system of a particular
+  * axis.
+  * Returns a single value or null if x is null.
+  */
+ Dygraph.prototype.toDomXCoord = function(x) {
+   if (x == null) {
+     return null;
+   };
    var area = this.plotter_.area;
-   if (x !== null) {
-     var xRange = this.xAxisRange();
-     ret[0] = area.x + (x - xRange[0]) / (xRange[1] - xRange[0]) * area.w;
-   }
+   var xRange = this.xAxisRange();
+   return area.x + (x - xRange[0]) / (xRange[1] - xRange[0]) * area.w;
+ }
  
-   if (y !== null) {
-     var yRange = this.yAxisRange(axis);
-     ret[1] = area.y + (yRange[1] - y) / (yRange[1] - yRange[0]) * area.h;
-   }
+ /**
+  * Convert from data x coordinates to canvas/div Y coordinate and optional
+  * axis. Uses the first axis by default.
+  *
+  * returns a single value or null if y is null.
+  */
+ Dygraph.prototype.toDomYCoord = function(y, axis) {
+   var pct = this.toPercentYCoord(y, axis);
  
-   return ret;
- };
+   if (pct == null) {
+     return null;
+   }
+   var area = this.plotter_.area;
+   return area.y + pct * area.h;
+ }
  
  /**
   * Convert from canvas/div coords to data coordinates.
   * If specified, do this conversion for the coordinate system of a particular
   * axis. Uses the first axis by default.
-  * Returns a two-element array: [X, Y]
+  * Returns a two-element array: [X, Y].
+  *
+  * Note: use toDataXCoord instead of toDataCoords(x, null) and use toDataYCoord
+  * instead of toDataCoords(null, y, axis).
   */
  Dygraph.prototype.toDataCoords = function(x, y, axis) {
-   var ret = [null, null];
-   var area = this.plotter_.area;
-   if (x !== null) {
-     var xRange = this.xAxisRange();
-     ret[0] = xRange[0] + (x - area.x) / area.w * (xRange[1] - xRange[0]);
+   return [ this.toDataXCoord(x), this.toDataYCoord(y, axis) ];
+ };
+ /**
+  * Convert from canvas/div x coordinate to data coordinate.
+  *
+  * If x is null, this returns null.
+  */
+ Dygraph.prototype.toDataXCoord = function(x) {
+   if (x == null) {
+     return null;
    }
  
-   if (y !== null) {
-     var yRange = this.yAxisRange(axis);
-     ret[1] = yRange[0] + (area.h - y) / area.h * (yRange[1] - yRange[0]);
+   var area = this.plotter_.area;
+   var xRange = this.xAxisRange();
+   return xRange[0] + (x - area.x) / area.w * (xRange[1] - xRange[0]);
+ };
+ /**
+  * Convert from canvas/div y coord to value.
+  *
+  * If y is null, this returns null.
+  * if axis is null, this uses the first axis.
+  */
+ Dygraph.prototype.toDataYCoord = function(y, axis) {
+   if (y == null) {
+     return null;
    }
  
-   return ret;
+   var area = this.plotter_.area;
+   var yRange = this.yAxisRange(axis);
+   if (typeof(axis) == "undefined") axis = 0;
+   if (!this.axes_[axis].logscale) {
+     return yRange[0] + (area.h - y) / area.h * (yRange[1] - yRange[0]);
+   } else {
+     // Computing the inverse of toDomCoord.
+     var pct = (y - area.y) / area.h
+     // Computing the inverse of toPercentYCoord. The function was arrived at with
+     // the following steps:
+     //
+     // Original calcuation:
+     // pct = (logr1 - Dygraph.log10(y)) / (logr1 - Dygraph.log10(yRange[0]));
+     //
+     // Move denominator to both sides:
+     // pct * (logr1 - Dygraph.log10(yRange[0])) = logr1 - Dygraph.log10(y);
+     //
+     // subtract logr1, and take the negative value.
+     // logr1 - (pct * (logr1 - Dygraph.log10(yRange[0]))) = Dygraph.log10(y);
+     //
+     // Swap both sides of the equation, and we can compute the log of the
+     // return value. Which means we just need to use that as the exponent in
+     // e^exponent.
+     // Dygraph.log10(y) = logr1 - (pct * (logr1 - Dygraph.log10(yRange[0])));
+     var logr1 = Dygraph.log10(yRange[1]);
+     var exponent = logr1 - (pct * (logr1 - Dygraph.log10(yRange[0])));
+     var value = Math.pow(Dygraph.LOG_SCALE, exponent);
+     return value;
+   }
  };
  
  /**
+  * Converts a y for an axis to a percentage from the top to the
+  * bottom of the div.
+  *
+  * If the coordinate represents a value visible on the canvas, then
+  * the value will be between 0 and 1, where 0 is the top of the canvas.
+  * However, this method will return values outside the range, as
+  * values can fall outside the canvas.
+  *
+  * If y is null, this returns null.
+  * if axis is null, this uses the first axis.
+  */
+ Dygraph.prototype.toPercentYCoord = function(y, axis) {
+   if (y == null) {
+     return null;
+   }
+   if (typeof(axis) == "undefined") axis = 0;
+   var area = this.plotter_.area;
+   var yRange = this.yAxisRange(axis);
+   var pct;
+   if (!this.axes_[axis].logscale) {
+     // yrange[1] - y is unit distance from the bottom.
+     // yrange[1] - yrange[0] is the scale of the range.
+     // (yRange[1] - y) / (yRange[1] - yRange[0]) is the % from the bottom.
+     pct = (yRange[1] - y) / (yRange[1] - yRange[0]);
+   } else {
+     var logr1 = Dygraph.log10(yRange[1]);
+     pct = (logr1 - Dygraph.log10(y)) / (logr1 - Dygraph.log10(yRange[0]));
+   }
+   return pct;
+ }
+ /**
   * Returns the number of columns (including the independent variable).
   */
  Dygraph.prototype.numColumns = function() {
@@@ -823,9 -928,17 +940,17 @@@ Dygraph.startPan = function(event, g, c
      var axis = g.axes_[i];
      var yRange = g.yAxisRange(i);
      // TODO(konigsberg): These values should be in |context|.
-     axis.dragValueRange = yRange[1] - yRange[0];
-     axis.initialTopValue = yRange[1];
+     // In log scale, initialTopValue, dragValueRange and unitsPerPixel are log scale.
+     if (axis.logscale) {
+       axis.initialTopValue = Dygraph.log10(yRange[1]);
+       axis.dragValueRange = Dygraph.log10(yRange[1]) - Dygraph.log10(yRange[0]);
+     } else {
+       axis.initialTopValue = yRange[1];
+       axis.dragValueRange = yRange[1] - yRange[0];
+     }
      axis.unitsPerPixel = axis.dragValueRange / (g.plotter_.area.h - 1);
+     // While calculating axes, set 2dpan.
      if (axis.valueWindow || axis.valueRange) context.is2DPan = true;
    }
  };
@@@ -851,10 -964,19 +976,19 @@@ Dygraph.movePan = function(event, g, co
      // Adjust each axis appropriately.
      for (var i = 0; i < g.axes_.length; i++) {
        var axis = g.axes_[i];
-       var maxValue = axis.initialTopValue +
-         (context.dragEndY - context.dragStartY) * axis.unitsPerPixel;
+       var pixelsDragged = context.dragEndY - context.dragStartY;
+       var unitsDragged = pixelsDragged * axis.unitsPerPixel;
+       // In log scale, maxValue and minValue are the logs of those values.
+       var maxValue = axis.initialTopValue + unitsDragged;
        var minValue = maxValue - axis.dragValueRange;
-       axis.valueWindow = [ minValue, maxValue ];
+       if (axis.logscale) {
+         axis.valueWindow = [ Math.pow(Dygraph.LOG_SCALE, minValue),
+                              Math.pow(Dygraph.LOG_SCALE, maxValue) ];
+       } else {
+         axis.valueWindow = [ minValue, maxValue ];
+       }
      }
    }
  
@@@ -1186,10 -1308,8 +1320,8 @@@ Dygraph.prototype.drawZoomRect_ = funct
  Dygraph.prototype.doZoomX_ = function(lowX, highX) {
    // Find the earliest and latest dates contained in this canvasx range.
    // Convert the call to date ranges of the raw data.
-   var r = this.toDataCoords(lowX, null);
-   var minDate = r[0];
-   r = this.toDataCoords(highX, null);
-   var maxDate = r[0];
+   var minDate = this.toDataXCoord(lowX);
+   var maxDate = this.toDataXCoord(highX);
    this.doZoomXDates_(minDate, maxDate);
  };
  
   */
  Dygraph.prototype.doZoomXDates_ = function(minDate, maxDate) {
    this.dateWindow_ = [minDate, maxDate];
 +  this.zoomed_x_ = true;
    this.drawGraph_();
    if (this.attr_("zoomCallback")) {
      this.attr_("zoomCallback")(minDate, maxDate, this.yAxisRanges());
@@@ -1226,17 -1345,15 +1358,17 @@@ Dygraph.prototype.doZoomY_ = function(l
    // coordinates increase as you go up the screen.
    var valueRanges = [];
    for (var i = 0; i < this.axes_.length; i++) {
-     var hi = this.toDataCoords(null, lowY, i);
-     var low = this.toDataCoords(null, highY, i);
-     this.axes_[i].valueWindow = [low[1], hi[1]];
-     valueRanges.push([low[1], hi[1]]);
+     var hi = this.toDataYCoord(lowY, i);
+     var low = this.toDataYCoord(highY, i);
+     this.axes_[i].valueWindow = [low, hi];
+     valueRanges.push([low, hi]);
    }
  
 +  this.zoomed_y_ = true;
    this.drawGraph_();
    if (this.attr_("zoomCallback")) {
      var xRange = this.xAxisRange();
 +    var yRange = this.yAxisRange();
      this.attr_("zoomCallback")(xRange[0], xRange[1], this.yAxisRanges());
    }
  };
@@@ -1264,8 -1381,6 +1396,8 @@@ Dygraph.prototype.doUnzoom_ = function(
    if (dirty) {
      // Putting the drawing operation before the callback because it resets
      // yAxisRange.
 +    this.zoomed_x_ = false;
 +    this.zoomed_y_ = false;
      this.drawGraph_();
      if (this.attr_("zoomCallback")) {
        var minDate = this.rawData_[0][0];
@@@ -1302,10 -1417,6 +1434,6 @@@ Dygraph.prototype.mouseMove_ = function
      idx = i;
    }
    if (idx >= 0) lastx = points[idx].xval;
-   // Check that you can really highlight the last day's data
-   var last = points[points.length-1];
-   if (last != null && canvasx > last.canvasx)
-     lastx = points[points.length-1].xval;
  
    // Extract the points we've selected
    this.selPoints_ = [];
@@@ -1813,10 -1924,75 +1941,75 @@@ Dygraph.dateTicker = function(startDate
    }
  };
  
+ // This is a list of human-friendly values at which to show tick marks on a log
+ // scale. It is k * 10^n, where k=1..9 and n=-39..+39, so:
+ // ..., 1, 2, 3, 4, 5, ..., 9, 10, 20, 30, ..., 90, 100, 200, 300, ...
+ // NOTE: this assumes that Dygraph.LOG_SCALE = 10.
+ Dygraph.PREFERRED_LOG_TICK_VALUES = function() {
+   var vals = [];
+   for (var power = -39; power <= 39; power++) {
+     var range = Math.pow(10, power);
+     for (var mult = 1; mult <= 9; mult++) {
+       var val = range * mult;
+       vals.push(val);
+     }
+   }
+   return vals;
+ }();
+ // val is the value to search for
+ // arry is the value over which to search
+ // if abs > 0, find the lowest entry greater than val
+ // if abs < 0, find the highest entry less than val
+ // if abs == 0, find the entry that equals val.
+ // Currently does not work when val is outside the range of arry's values.
+ Dygraph.binarySearch = function(val, arry, abs, low, high) {
+   if (low == null || high == null) {
+     low = 0;
+     high = arry.length - 1;
+   }
+   if (low > high) {
+     return -1;
+   }
+   if (abs == null) {
+     abs = 0;
+   }
+   var validIndex = function(idx) {
+     return idx >= 0 && idx < arry.length;
+   }
+   var mid = parseInt((low + high) / 2);
+   var element = arry[mid];
+   if (element == val) {
+     return mid;
+   }
+   if (element > val) {
+     if (abs > 0) {
+       // Accept if element > val, but also if prior element < val.
+       var idx = mid - 1;
+       if (validIndex(idx) && arry[idx] < val) {
+         return mid;
+       }
+     }
+     return Dygraph.binarySearch(val, arry, abs, low, mid - 1);
+   }
+   if (element < val) {
+     if (abs < 0) {
+       // Accept if element < val, but also if prior element > val.
+       var idx = mid + 1;
+       if (validIndex(idx) && arry[idx] > val) {
+         return mid;
+       }
+     }
+     return Dygraph.binarySearch(val, arry, abs, mid + 1, high);
+   }
+ };
  /**
   * Add ticks when the x axis has numbers on it (instead of dates)
-  * @param {Number} startDate Start of the date window (millis since epoch)
-  * @param {Number} endDate End of the date window (millis since epoch)
+  * TODO(konigsberg): Update comment.
+  *
+  * @param {Number} minV minimum value
+  * @param {Number} maxV maximum value
   * @param self
   * @param {function} attribute accessor function.
   * @return {Array.<Object>} Array of {label, value} tuples.
@@@ -1834,43 -2010,89 +2027,89 @@@ Dygraph.numericTicks = function(minV, m
        ticks.push({v: vals[i]});
      }
    } else {
-     // Basic idea:
-     // Try labels every 1, 2, 5, 10, 20, 50, 100, etc.
-     // Calculate the resulting tick spacing (i.e. this.height_ / nTicks).
-     // The first spacing greater than pixelsPerYLabel is what we use.
-     // TODO(danvk): version that works on a log scale.
-     if (attr("labelsKMG2")) {
-       var mults = [1, 2, 4, 8];
-     } else {
-       var mults = [1, 2, 5];
+     if (axis_props && attr("logscale")) {
+       var pixelsPerTick = attr('pixelsPerYLabel');
+       // NOTE(konigsberg): Dan, should self.height_ be self.plotter_.area.h?
+       var nTicks  = Math.floor(self.height_ / pixelsPerTick);
+       var minIdx = Dygraph.binarySearch(minV, Dygraph.PREFERRED_LOG_TICK_VALUES, 1);
+       var maxIdx = Dygraph.binarySearch(maxV, Dygraph.PREFERRED_LOG_TICK_VALUES, -1);
+       if (minIdx == -1) {
+         minIdx = 0;
+       }
+       if (maxIdx == -1) {
+         maxIdx = Dygraph.PREFERRED_LOG_TICK_VALUES.length - 1;
+       }
+       // Count the number of tick values would appear, if we can get at least
+       // nTicks / 4 accept them.
+       var lastDisplayed = null;
+       if (maxIdx - minIdx >= nTicks / 4) {
+         var axisId = axis_props.yAxisId;
+         for (var idx = maxIdx; idx >= minIdx; idx--) {
+           var tickValue = Dygraph.PREFERRED_LOG_TICK_VALUES[idx];
+           var domCoord = axis_props.g.toDomYCoord(tickValue, axisId);
+           var tick = { v: tickValue };
+           if (lastDisplayed == null) {
+             lastDisplayed = {
+               tickValue : tickValue,
+               domCoord : domCoord
+             };
+           } else {
+             if (domCoord - lastDisplayed.domCoord >= pixelsPerTick) {
+               lastDisplayed = {
+                 tickValue : tickValue,
+                 domCoord : domCoord
+               };
+             } else {
+               tick.label = "";
+             }
+           }
+           ticks.push(tick);
+         }
+         // Since we went in backwards order.
+         ticks.reverse();
+       }
      }
-     var scale, low_val, high_val, nTicks;
-     // TODO(danvk): make it possible to set this for x- and y-axes independently.
-     var pixelsPerTick = attr('pixelsPerYLabel');
-     for (var i = -10; i < 50; i++) {
+     // ticks.length won't be 0 if the log scale function finds values to insert.
+     if (ticks.length == 0) {
+       // Basic idea:
+       // Try labels every 1, 2, 5, 10, 20, 50, 100, etc.
+       // Calculate the resulting tick spacing (i.e. this.height_ / nTicks).
+       // The first spacing greater than pixelsPerYLabel is what we use.
+       // TODO(danvk): version that works on a log scale.
        if (attr("labelsKMG2")) {
-         var base_scale = Math.pow(16, i);
+         var mults = [1, 2, 4, 8];
        } else {
-         var base_scale = Math.pow(10, i);
+         var mults = [1, 2, 5];
        }
-       for (var j = 0; j < mults.length; j++) {
-         scale = base_scale * mults[j];
-         low_val = Math.floor(minV / scale) * scale;
-         high_val = Math.ceil(maxV / scale) * scale;
-         nTicks = Math.abs(high_val - low_val) / scale;
-         var spacing = self.height_ / nTicks;
-         // wish I could break out of both loops at once...
+       var scale, low_val, high_val, nTicks;
+       // TODO(danvk): make it possible to set this for x- and y-axes independently.
+       var pixelsPerTick = attr('pixelsPerYLabel');
+       for (var i = -10; i < 50; i++) {
+         if (attr("labelsKMG2")) {
+           var base_scale = Math.pow(16, i);
+         } else {
+           var base_scale = Math.pow(10, i);
+         }
+         for (var j = 0; j < mults.length; j++) {
+           scale = base_scale * mults[j];
+           low_val = Math.floor(minV / scale) * scale;
+           high_val = Math.ceil(maxV / scale) * scale;
+           nTicks = Math.abs(high_val - low_val) / scale;
+           var spacing = self.height_ / nTicks;
+           // wish I could break out of both loops at once...
+           if (spacing > pixelsPerTick) break;
+         }
          if (spacing > pixelsPerTick) break;
        }
-       if (spacing > pixelsPerTick) break;
-     }
  
-     // Construct the set of ticks.
-     // Allow reverse y-axis if it's explicitly requested.
-     if (low_val > high_val) scale *= -1;
-     for (var i = 0; i < nTicks; i++) {
-       var tickV = low_val + i * scale;
-       ticks.push( {v: tickV} );
+       // Construct the set of ticks.
+       // Allow reverse y-axis if it's explicitly requested.
+       if (low_val > high_val) scale *= -1;
+       for (var i = 0; i < nTicks; i++) {
+         var tickV = low_val + i * scale;
+         ticks.push( {v: tickV} );
+       }
      }
    }
  
    }
    var formatter = attr('yAxisLabelFormatter') ? attr('yAxisLabelFormatter') : attr('yValueFormatter'); 
  
+   // Add labels to the ticks.
    for (var i = 0; i < ticks.length; i++) {
-     var tickV = ticks[i].v;
-     var absTickV = Math.abs(tickV);
-     var label;
-     if (formatter != undefined) {
-       label = formatter(tickV);
-     } else {
-       label = Dygraph.round_(tickV, 2);
-     }
-     if (k_labels.length) {
-       // Round up to an appropriate unit.
-       var n = k*k*k*k;
-       for (var j = 3; j >= 0; j--, n /= k) {
-         if (absTickV >= n) {
-           label = Dygraph.round_(tickV / n, 1) + k_labels[j];
-           break;
+     if (ticks[i].label == null) {
+       var tickV = ticks[i].v;
+       var absTickV = Math.abs(tickV);
+       var label;
+       if (formatter != undefined) {
+         label = formatter(tickV);
+       } else {
+         label = Dygraph.round_(tickV, 2);
+       }
+       if (k_labels.length) {
+         // Round up to an appropriate unit.
+         var n = k*k*k*k;
+         for (var j = 3; j >= 0; j--, n /= k) {
+           if (absTickV >= n) {
+             label = Dygraph.round_(tickV / n, 1) + k_labels[j];
+             break;
+           }
          }
        }
+       ticks[i].label = label;
      }
-     ticks[i].label = label;
    }
    return ticks;
  };
@@@ -1983,7 -2208,6 +2225,6 @@@ Dygraph.prototype.predraw_ = function(
  };
  
  /**
- =======
   * Update the graph with new data. This method is called when the viewing area
   * has changed. If the underlying data or options have changed, predraw_ will
   * be called before drawGraph_ is called.
@@@ -2015,12 -2239,24 +2256,24 @@@ Dygraph.prototype.drawGraph_ = function
  
      var seriesName = this.attr_("labels")[i];
      var connectSeparatedPoints = this.attr_('connectSeparatedPoints', i);
+     var logScale = this.attr_('logscale', i);
  
      var series = [];
      for (var j = 0; j < data.length; j++) {
-       if (data[j][i] != null || !connectSeparatedPoints) {
-         var date = data[j][0];
-         series.push([date, data[j][i]]);
+       var date = data[j][0];
+       var point = data[j][i];
+       if (logScale) {
+         // On the log scale, points less than zero do not exist.
+         // This will create a gap in the chart. Note that this ignores
+         // connectSeparatedPoints.
+         if (point <= 0) {
+           point = null;
+         }
+         series.push([date, point]);
+       } else {
+         if (point != null || !connectSeparatedPoints) {
+           series.push([date, point]);
+         }
        }
      }
  
      this.layout_.addDataset(this.attr_("labels")[i], datasets[i]);
    }
  
 -  // TODO(danvk): this method doesn't need to return anything.
 -  var out = this.computeYAxisRanges_(extremes);
 -  var axes = out[0];
 -  var seriesToAxisMap = out[1];
 -  this.layout_.updateOptions( { yAxes: axes,
 -                                seriesToAxisMap: seriesToAxisMap
 -                              } );
 -
 +  if (datasets.length > 0) {
 +    // TODO(danvk): this method doesn't need to return anything.
 +    var out = this.computeYAxisRanges_(extremes);
 +    var axes = out[0];
 +    var seriesToAxisMap = out[1];
 +    this.layout_.updateOptions( { yAxes: axes,
 +                                  seriesToAxisMap: seriesToAxisMap
 +                                } );
 +  }
    this.addXTicks_();
  
 +  // Save the X axis zoomed status as the updateOptions call will tend to set it errorneously
 +  var tmp_zoomed_x = this.zoomed_x_;
    // Tell PlotKit to use this new data and render itself
    this.layout_.updateOptions({dateWindow: this.dateWindow_});
 +  this.zoomed_x_ = tmp_zoomed_x;
    this.layout_.evaluateWithError();
    this.plotter_.clear();
    this.plotter_.render();
   *   indices are into the axes_ array.
   */
  Dygraph.prototype.computeYAxes_ = function() {
-   this.axes_ = [{ yAxisId: 0 }];  // always have at least one y-axis.
 +  var valueWindows;
 +  if (this.axes_ != undefined) {
 +    // Preserve valueWindow settings.
 +    valueWindows = [];
 +    for (var index = 0; index < this.axes_.length; index++) {
 +      valueWindows.push(this.axes_[index].valueWindow);
 +    }
 +  }
 +
+   this.axes_ = [{ yAxisId : 0, g : this }];  // always have at least one y-axis.
    this.seriesToAxisMap_ = {};
  
    // Get a list of series names.
      'pixelsPerYLabel',
      'yAxisLabelWidth',
      'axisLabelFontSize',
-     'axisTickSize'
+     'axisTickSize',
+     'logscale'
    ];
  
    // Copy global axis options over to the first axis.
        Dygraph.update(opts, { valueRange: null });  // shouldn't inherit this.
        var yAxisId = this.axes_.length;
        opts.yAxisId = yAxisId;
+       opts.g = this;
        Dygraph.update(opts, axis);
        this.axes_.push(opts);
        this.seriesToAxisMap_[seriesName] = yAxisId;
      if (vis[i - 1]) seriesToAxisFiltered[s] = this.seriesToAxisMap_[s];
    }
    this.seriesToAxisMap_ = seriesToAxisFiltered;
 +
 +  if (valueWindows != undefined) {
 +    // Restore valueWindow settings.
 +    for (var index = 0; index < valueWindows.length; index++) {
 +      this.axes_[index].valueWindow = valueWindows[index];
 +    }
 +  }
  };
  
  /**
@@@ -2285,18 -2503,26 +2540,26 @@@ Dygraph.prototype.computeYAxisRanges_ 
        var span = maxY - minY;
        // special case: if we have no sense of scale, use +/-10% of the sole value.
        if (span == 0) { span = maxY; }
-       var maxAxisY = maxY + 0.1 * span;
-       var minAxisY = minY - 0.1 * span;
  
-       // Try to include zero and make it minAxisY (or maxAxisY) if it makes sense.
-       if (!this.attr_("avoidMinZero")) {
-         if (minAxisY < 0 && minY >= 0) minAxisY = 0;
-         if (maxAxisY > 0 && maxY <= 0) maxAxisY = 0;
-       }
+       var maxAxisY;
+       var minAxisY;
+       if (axis.logscale) {
+         var maxAxisY = maxY + 0.1 * span;
+         var minAxisY = minY;
+       } else {
+         var maxAxisY = maxY + 0.1 * span;
+         var minAxisY = minY - 0.1 * span;
  
-       if (this.attr_("includeZero")) {
-         if (maxY < 0) maxAxisY = 0;
-         if (minY > 0) minAxisY = 0;
+         // Try to include zero and make it minAxisY (or maxAxisY) if it makes sense.
+         if (!this.attr_("avoidMinZero")) {
+           if (minAxisY < 0 && minY >= 0) minAxisY = 0;
+           if (maxAxisY > 0 && maxY <= 0) maxAxisY = 0;
+         }
+         if (this.attr_("includeZero")) {
+           if (maxY < 0) maxAxisY = 0;
+           if (minY > 0) minAxisY = 0;
+         }
        }
  
        axis.computedValueRange = [minAxisY, maxAxisY];
@@@ -2510,7 -2736,7 +2773,7 @@@ Dygraph.dateParser = function(dateStr, 
   */
  Dygraph.prototype.detectTypeFromString_ = function(str) {
    var isDate = false;
-   if (str.indexOf('-') >= 0 ||
+   if (str.indexOf('-') > 0 ||
        str.indexOf('/') >= 0 ||
        isNaN(parseFloat(str))) {
      isDate = true;
@@@ -2923,12 -3149,6 +3186,12 @@@ Dygraph.prototype.updateOptions = funct
    }
    if ('dateWindow' in attrs) {
      this.dateWindow_ = attrs.dateWindow;
 +    if (!('noZoomFlagChange' in attrs)) {
 +      this.zoomed_x_ = attrs.dateWindow != null;
 +    }
 +  }
 +  if ('valueRange' in attrs && !('noZoomFlagChange' in attrs)) {
 +    this.zoomed_y_ = attrs.valueRange != null;
    }
  
    // TODO(danvk): validate per-series options.