Merge branch 'master' into xlog
authorRobert Konigsberg <konigsberg@google.com>
Wed, 23 Apr 2014 12:47:08 +0000 (08:47 -0400)
committerRobert Konigsberg <konigsberg@google.com>
Wed, 23 Apr 2014 12:47:08 +0000 (08:47 -0400)
Conflicts:
dygraph-layout.js

1  2 
dygraph-interaction-model.js
dygraph-layout.js
dygraph.js

@@@ -38,18 -38,12 +38,18 @@@ Dygraph.Interaction.startPan = function
    var i, axis;
    context.isPanning = true;
    var xRange = g.xAxisRange();
 -  context.dateRange = xRange[1] - xRange[0];
 -  context.initialLeftmostDate = xRange[0];
 +
 +  if (g.getOptionForAxis("logscale", 'x')) {
 +    context.initialLeftmostDate = Dygraph.log10(xRange[0]);
 +    context.dateRange = Dygraph.log10(xRange[1]) - Dygraph.log10(xRange[0]);
 +  } else {
 +    context.initialLeftmostDate = xRange[0];    
 +    context.dateRange = xRange[1] - xRange[0];
 +  }
    context.xUnitsPerPixel = context.dateRange / (g.plotter_.area.w - 1);
  
-   if (g.attr_("panEdgeFraction")) {
-     var maxXPixelsToDraw = g.width_ * g.attr_("panEdgeFraction");
+   if (g.getNumericOption("panEdgeFraction")) {
+     var maxXPixelsToDraw = g.width_ * g.getNumericOption("panEdgeFraction");
      var xExtremes = g.xAxisExtremes(); // I REALLY WANT TO CALL THIS xTremes!
  
      var boundedLeftX = g.toDomXCoord(xExtremes[0]) - maxXPixelsToDraw;
@@@ -60,7 -54,7 +60,7 @@@
      context.boundedDates = [boundedLeftDate, boundedRightDate];
  
      var boundedValues = [];
-     var maxYPixelsToDraw = g.height_ * g.attr_("panEdgeFraction");
+     var maxYPixelsToDraw = g.height_ * g.getNumericOption("panEdgeFraction");
  
      for (i = 0; i < g.axes_.length; i++) {
        axis = g.axes_[i];
   *     context.
   */
  Dygraph.Interaction.movePan = function(event, g, context) {
-   context.dragEndX = g.dragGetX_(event, context);
-   context.dragEndY = g.dragGetY_(event, context);
+   context.dragEndX = Dygraph.dragGetX_(event, context);
+   context.dragEndY = Dygraph.dragGetY_(event, context);
  
    var minDate = context.initialLeftmostDate -
      (context.dragEndX - context.dragStartX) * context.xUnitsPerPixel;
      }
    }
  
 -  g.dateWindow_ = [minDate, maxDate];
 +  if (g.getOptionForAxis("logscale", 'x')) {
 +    g.dateWindow_ = [ Math.pow(Dygraph.LOG_SCALE, minDate),
 +                      Math.pow(Dygraph.LOG_SCALE, maxDate) ];
 +  } else {
 +    g.dateWindow_ = [minDate, maxDate];    
 +  }
  
    // y-axis scaling is automatic unless this is a full 2D pan.
    if (context.is2DPan) {
            minValue = maxValue - axis_data.dragValueRange;
          }
        }
 -      var logscale = g.attributes_.getForAxis("logscale", i);
 -      if (logscale) {
 +      if (g.attributes_.getForAxis("logscale", i)) {
          axis.valueWindow = [ Math.pow(Dygraph.LOG_SCALE, minValue),
                               Math.pow(Dygraph.LOG_SCALE, maxValue) ];
        } else {
   *     context.
   */
  Dygraph.Interaction.endPan = function(event, g, context) {
-   context.dragEndX = g.dragGetX_(event, context);
-   context.dragEndY = g.dragGetY_(event, context);
+   context.dragEndX = Dygraph.dragGetX_(event, context);
+   context.dragEndY = Dygraph.dragGetY_(event, context);
  
    var regionWidth = Math.abs(context.dragEndX - context.dragStartX);
    var regionHeight = Math.abs(context.dragEndY - context.dragStartY);
@@@ -256,8 -246,8 +256,8 @@@ Dygraph.Interaction.startZoom = functio
   */
  Dygraph.Interaction.moveZoom = function(event, g, context) {
    context.zoomMoved = true;
-   context.dragEndX = g.dragGetX_(event, context);
-   context.dragEndY = g.dragGetY_(event, context);
+   context.dragEndX = Dygraph.dragGetX_(event, context);
+   context.dragEndY = Dygraph.dragGetY_(event, context);
  
    var xDelta = Math.abs(context.dragStartX - context.dragEndX);
    var yDelta = Math.abs(context.dragStartY - context.dragEndY);
   * @param {Object} context
   */
  Dygraph.Interaction.treatMouseOpAsClick = function(g, event, context) {
-   var clickCallback = g.attr_('clickCallback');
-   var pointClickCallback = g.attr_('pointClickCallback');
+   var clickCallback = g.getFunctionOption('clickCallback');
+   var pointClickCallback = g.getFunctionOption('pointClickCallback');
  
    var selectedPoint = null;
  
      }
  
      // Allow any click within two pixels of the dot.
-     var radius = g.attr_('highlightCircleSize') + 2;
+     var radius = g.getNumericOption('highlightCircleSize') + 2;
      if (closestDistance <= radius * radius) {
        selectedPoint = g.selPoints_[closestIdx];
      }
   */
  Dygraph.Interaction.endZoom = function(event, g, context) {
    context.isZooming = false;
-   context.dragEndX = g.dragGetX_(event, context);
-   context.dragEndY = g.dragGetY_(event, context);
+   context.dragEndX = Dygraph.dragGetX_(event, context);
+   context.dragEndY = Dygraph.dragGetY_(event, context);
    var regionWidth = Math.abs(context.dragEndX - context.dragStartX);
    var regionHeight = Math.abs(context.dragEndY - context.dragStartY);
  
@@@ -533,9 -523,9 +533,9 @@@ Dygraph.Interaction.moveTouch = functio
    g.drawGraph_(false);
  
    // We only call zoomCallback on zooms, not pans, to mirror desktop behavior.
-   if (didZoom && touches.length > 1 && g.attr_('zoomCallback')) {
+   if (didZoom && touches.length > 1 && g.getFunctionOption('zoomCallback')) {
      var viewWindow = g.xAxisRange();
-     g.attr_("zoomCallback")(viewWindow[0], viewWindow[1], g.yAxisRanges());
+     g.getFunctionOption("zoomCallback")(viewWindow[0], viewWindow[1], g.yAxisRanges());
    }
  };
  
@@@ -655,8 -645,8 +655,8 @@@ Dygraph.Interaction.nonInteractiveModel
    },
    mouseup: function(event, g, context) {
      // TODO(danvk): this logic is repeated in Dygraph.Interaction.endZoom
-     context.dragEndX = g.dragGetX_(event, context);
-     context.dragEndY = g.dragGetY_(event, context);
+     context.dragEndX = Dygraph.dragGetX_(event, context);
+     context.dragEndY = Dygraph.dragGetY_(event, context);
      var regionWidth = Math.abs(context.dragEndX - context.dragStartX);
      var regionHeight = Math.abs(context.dragEndY - context.dragStartY);
  
diff --combined dygraph-layout.js
@@@ -149,14 -149,14 +149,14 @@@ DygraphLayout.prototype.setAnnotations 
    for (var i = 0; i < ann.length; i++) {
      var a = {};
      if (!ann[i].xval && ann[i].x === undefined) {
-       this.dygraph_.error("Annotations must have an 'x' property");
+       Dygraph.error("Annotations must have an 'x' property");
        return;
      }
      if (ann[i].icon &&
          !(ann[i].hasOwnProperty('width') &&
            ann[i].hasOwnProperty('height'))) {
-       this.dygraph_.error("Must set width and height when setting " +
-                           "annotation.icon property");
+       Dygraph.error("Must set width and height when setting " +
+                     "annotation.icon property");
        return;
      }
      Dygraph.update(a, ann[i]);
@@@ -175,7 -175,6 +175,7 @@@ DygraphLayout.prototype.setYAxes = func
  };
  
  DygraphLayout.prototype.evaluate = function() {
 +  this._xAxis = {};
    this._evaluateLimits();
    this._evaluateLineCharts();
    this._evaluateLineTicks();
  
  DygraphLayout.prototype._evaluateLimits = function() {
    var xlimits = this.dygraph_.xAxisRange();
 -  this.minxval = xlimits[0];
 -  this.maxxval = xlimits[1];
 +  this._xAxis.minxval = xlimits[0];
 +  this._xAxis.maxxval = xlimits[1];
    var xrange = xlimits[1] - xlimits[0];
 -  this.xscale = (xrange !== 0 ? 1 / xrange : 1.0);
 +  this._xAxis.xscale = (xrange !== 0 ? 1 / xrange : 1.0);
  
 +  if (this.dygraph_.getOptionForAxis("logscale", 'x')) {
 +    this._xAxis.xlogrange = Dygraph.log10(this._xAxis.maxxval) - Dygraph.log10(this._xAxis.minxval);
 +    this._xAxis.xlogscale = (this._xAxis.xlogrange !== 0 ? 1.0 / this._xAxis.xlogrange : 1.0);
 +  }
    for (var i = 0; i < this.yAxes_.length; i++) {
      var axis = this.yAxes_[i];
      axis.minyval = axis.computedValueRange[0];
      axis.yrange = axis.maxyval - axis.minyval;
      axis.yscale = (axis.yrange !== 0 ? 1.0 / axis.yrange : 1.0);
  
 -    if (axis.g.getOption("logscale")) {
 +    if (this.dygraph_.getOption("logscale")) {
        axis.ylogrange = Dygraph.log10(axis.maxyval) - Dygraph.log10(axis.minyval);
        axis.ylogscale = (axis.ylogrange !== 0 ? 1.0 / axis.ylogrange : 1.0);
        if (!isFinite(axis.ylogrange) || isNaN(axis.ylogrange)) {
-         axis.g.error('axis ' + i + ' of graph at ' + axis.g +
-             ' can\'t be displayed in log scale for range [' +
-             axis.minyval + ' - ' + axis.maxyval + ']');
+         Dygraph.error('axis ' + i + ' of graph at ' + axis.g +
+                       ' can\'t be displayed in log scale for range [' +
+                       axis.minyval + ' - ' + axis.maxyval + ']');
        }
      }
    }
  };
  
- DygraphLayout._calcXNormal = function(value, axis, logscale) {
++DygraphLayout.calcXNormal_ = function(value, axis, logscale) {
 +  if (logscale) {
 +    return ((Dygraph.log10(value) - Dygraph.log10(axis.minxval)) * axis.xlogscale);
 +  } else {
 +    return (value - axis.minxval) * axis.xscale;
 +  }
 +};
 +
- DygraphLayout._calcYNormal = function(axis, value, logscale) {
+ /**
+  * @param {DygraphAxisType} axis
+  * @param {number} value
+  * @param {boolean} logscale
+  * @return {number}
+  */
+ DygraphLayout.calcYNormal_ = function(axis, value, logscale) {
    if (logscale) {
-     return 1.0 - ((Dygraph.log10(value) - Dygraph.log10(axis.minyval)) * axis.ylogscale);
+     var x = 1.0 - ((Dygraph.log10(value) - Dygraph.log10(axis.minyval)) * axis.ylogscale);
+     return isFinite(x) ? x : NaN;  // shim for v8 issue; see pull request 276
    } else {
      return 1.0 - ((value - axis.minyval) * axis.yscale);
    }
  
  DygraphLayout.prototype._evaluateLineCharts = function() {
    var isStacked = this.dygraph_.getOption("stackedGraph");
 +  var isLogscaleForX = this.dygraph_.getOptionForAxis("logscale", 'x');
  
    for (var setIdx = 0; setIdx < this.points.length; setIdx++) {
      var points = this.points[setIdx];
        var point = points[j];
  
        // Range from 0-1 where 0 represents left and 1 represents right.
-       point.x = DygraphLayout._calcXNormal(point.xval, this._xAxis, isLogscaleForX);
 -      point.x = (point.xval - this.minxval) * this.xscale;
++      point.x = DygraphLayout.calcXNormal_(point.xval, this._xAxis, isLogscaleForX);
        // Range from 0-1 where 0 represents top and 1 represents bottom
        var yval = point.yval;
        if (isStacked) {
-         point.y_stacked = DygraphLayout._calcYNormal(
+         point.y_stacked = DygraphLayout.calcYNormal_(
              axis, point.yval_stacked, logscale);
          if (yval !== null && !isNaN(yval)) {
            yval = point.yval_stacked;
            point.yval = NaN;
          }
        }
-       point.y = DygraphLayout._calcYNormal(axis, yval, logscale);
+       point.y = DygraphLayout.calcYNormal_(axis, yval, logscale);
      }
  
      this.dygraph_.dataHandler_.onLineEvaluated(points, axis, logscale);
    }
  };
  
- /**
-  * Optimized replacement for parseFloat, which was way too slow when almost
-  * all values were type number, with few edge cases, none of which were strings.
-  */
- DygraphLayout.parseFloat_ = function(val) {
-   // parseFloat(null) is NaN
-   if (val === null) {
-     return NaN;
-   }
-   // Assume it's a number or NaN. If it's something else, I'll be shocked.
-   return val;
- };
  DygraphLayout.prototype._evaluateLineTicks = function() {
    var i, tick, label, pos;
    this.xticks = [];
    for (i = 0; i < this.xTicks_.length; i++) {
      tick = this.xTicks_[i];
      label = tick.label;
 -    pos = this.xscale * (tick.v - this.minxval);
 +    pos = this.dygraph_.toPercentXCoord(tick.v);
-     if ((pos >= 0.0) && (pos <= 1.0)) {
+     if ((pos >= 0.0) && (pos < 1.0)) {
        this.xticks.push([pos, label]);
      }
    }
        tick = axis.ticks[j];
        label = tick.label;
        pos = this.dygraph_.toPercentYCoord(tick.v, i);
-       if ((pos >= 0.0) && (pos <= 1.0)) {
+       if ((pos > 0.0) && (pos <= 1.0)) {
          this.yticks.push([i, pos, label]);
        }
      }
diff --combined dygraph.js
@@@ -72,7 -72,7 +72,7 @@@ var Dygraph = function(div, data, opts
      // Old versions of dygraphs took in the series labels as a constructor
      // parameter. This doesn't make sense anymore, but it's easy to continue
      // to support this usage.
-     this.warn("Using deprecated four-argument dygraph constructor");
+     Dygraph.warn("Using deprecated four-argument dygraph constructor");
      this.__old_init__(div, data, opts, opt_fourth_param);
    } else {
      this.__init__(div, data, opts);
  Dygraph.NAME = "Dygraph";
  Dygraph.VERSION = "1.0.1";
  Dygraph.__repr__ = function() {
-   return "[" + this.NAME + " " + this.VERSION + "]";
+   return "[" + Dygraph.NAME + " " + Dygraph.VERSION + "]";
  };
  
  /**
   * Returns information about the Dygraph class.
   */
  Dygraph.toString = function() {
-   return this.__repr__();
+   return Dygraph.__repr__();
  };
  
  // Various default values
@@@ -112,9 -112,9 +112,9 @@@ Dygraph.KMG2_SMALL_LABELS = [ 'm', 'u'
   * @private
   * Return a string version of a number. This respects the digitsAfterDecimal
   * and maxNumberWidth options.
-  * @param {Number} x The number to be formatted
+  * @param {number} x The number to be formatted
   * @param {Dygraph} opts An options view
-  * @param {String} name The name of the point's data series
+  * @param {string} name The name of the point's data series
   * @param {Dygraph} g The dygraph object
   */
  Dygraph.numberValueFormatter = function(x, opts, pt, g) {
@@@ -193,46 -193,32 +193,32 @@@ Dygraph.numberAxisLabelFormatter = func
  };
  
  /**
-  * Convert a JS date (millis since epoch) to YYYY/MM/DD
-  * @param {Number} date The JavaScript date (ms since epoch)
-  * @return {String} A date of the form "YYYY/MM/DD"
+  * @type {!Array.<string>}
   * @private
+  * @constant
   */
- Dygraph.dateString_ = function(date) {
-   var zeropad = Dygraph.zeropad;
-   var d = new Date(date);
+ Dygraph.SHORT_MONTH_NAMES_ = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];
  
-   // Get the year:
-   var year = "" + d.getFullYear();
-   // Get a 0 padded month string
-   var month = zeropad(d.getMonth() + 1);  //months are 0-offset, sigh
-   // Get a 0 padded day string
-   var day = zeropad(d.getDate());
-   var ret = "";
-   var frac = d.getHours() * 3600 + d.getMinutes() * 60 + d.getSeconds();
-   if (frac) ret = " " + Dygraph.hmsString_(date);
-   return year + "/" + month + "/" + day + ret;
- };
  
  /**
   * Convert a JS date to a string appropriate to display on an axis that
   * is displaying values at the stated granularity.
   * @param {Date} date The date to format
-  * @param {Number} granularity One of the Dygraph granularity constants
-  * @return {String} The formatted date
+  * @param {number} granularity One of the Dygraph granularity constants
+  * @return {string} The formatted date
   * @private
   */
  Dygraph.dateAxisFormatter = function(date, granularity) {
    if (granularity >= Dygraph.DECADAL) {
-     return date.strftime('%Y');
+     return '' + date.getFullYear();
    } else if (granularity >= Dygraph.MONTHLY) {
-     return date.strftime('%b %y');
+     return Dygraph.SHORT_MONTH_NAMES_[date.getMonth()] + ' ' + date.getFullYear();
    } else {
      var frac = date.getHours() * 3600 + date.getMinutes() * 60 + date.getSeconds() + date.getMilliseconds();
      if (frac === 0 || granularity >= Dygraph.DAILY) {
-       return new Date(date.getTime() + 3600*1000).strftime('%d%b');
+       // e.g. '21Jan' (%d%b)
+       var nd = new Date(date.getTime() + 3600*1000);
+       return Dygraph.zeropad(nd.getDate()) + Dygraph.SHORT_MONTH_NAMES_[nd.getMonth()];
      } else {
        return Dygraph.hmsString_(date.getTime());
      }
@@@ -407,7 -393,7 +393,7 @@@ Dygraph.prototype.__old_init__ = functi
   * and context &lt;canvas&gt; inside of it. See the constructor for details.
   * on the parameters.
   * @param {Element} div the Element to render the graph into.
-  * @param {String | Function} file Source data
+  * @param {string | Function} file Source data
   * @param {Object} attrs Miscellaneous other options
   * @private
   */
@@@ -599,6 -585,22 +585,22 @@@ Dygraph.prototype.cascadeEvents_ = func
  };
  
  /**
+  * Fetch a plugin instance of a particular class. Only for testing.
+  * @private
+  * @param {!Class} type The type of the plugin.
+  * @return {Object} Instance of the plugin, or null if there is none.
+  */
+ Dygraph.prototype.getPluginInstance_ = function(type) {
+   for (var i = 0; i < this.plugins_.length; i++) {
+     var p = this.plugins_[i];
+     if (p.plugin instanceof type) {
+       return p.plugin;
+     }
+   }
+   return null;
+ };
+ /**
   * Returns the zoomed status of the chart for one or both axes.
   *
   * Axis is an optional parameter. Can be set to 'x' or 'y'.
@@@ -630,8 -632,8 +632,8 @@@ Dygraph.prototype.toString = function(
   * Returns the value of an option. This may be set by the user (either in the
   * constructor or by calling updateOptions) or by dygraphs, and may be set to a
   * per-series value.
-  * @param { String } name The name of the option, e.g. 'rollPeriod'.
-  * @param { String } [seriesName] The name of the series to which the option
+  * @param {string} name The name of the option, e.g. 'rollPeriod'.
+  * @param {string} [seriesName] The name of the series to which the option
   * will be applied. If no per-series value of this option is available, then
   * the global value is returned. This is optional.
   * @return { ... } The value of the option.
  Dygraph.prototype.attr_ = function(name, seriesName) {
  // <REMOVE_FOR_COMBINED>
    if (typeof(Dygraph.OPTIONS_REFERENCE) === 'undefined') {
-     this.error('Must include options reference JS for testing');
+     Dygraph.error('Must include options reference JS for testing');
    } else if (!Dygraph.OPTIONS_REFERENCE.hasOwnProperty(name)) {
-     this.error('Dygraphs is using property ' + name + ', which has no entry ' +
-                'in the Dygraphs.OPTIONS_REFERENCE listing.');
+     Dygraph.error('Dygraphs is using property ' + name + ', which has no ' +
+                   'entry in the Dygraphs.OPTIONS_REFERENCE listing.');
      // Only log this error once.
      Dygraph.OPTIONS_REFERENCE[name] = true;
    }
   * dygraphs will remain in a consistent state. If you want to modify an option,
   * use updateOptions() instead.
   *
-  * @param { String } name The name of the option (e.g. 'strokeWidth')
-  * @param { String } [opt_seriesName] Series name to get per-series values.
-  * @return { ... } The value of the option.
+  * @param {string} name The name of the option (e.g. 'strokeWidth')
+  * @param {string=} opt_seriesName Series name to get per-series values.
+  * @return {*} The value of the option.
   */
  Dygraph.prototype.getOption = function(name, opt_seriesName) {
    return this.attr_(name, opt_seriesName);
  };
  
+ /**
+  * Like getOption(), but specifically returns a number.
+  * This is a convenience function for working with the Closure Compiler.
+  * @param {string} name The name of the option (e.g. 'strokeWidth')
+  * @param {string=} opt_seriesName Series name to get per-series values.
+  * @return {number} The value of the option.
+  * @private
+  */
+ Dygraph.prototype.getNumericOption = function(name, opt_seriesName) {
+   return /** @type{number} */(this.getOption(name, opt_seriesName));
+ };
+ /**
+  * Like getOption(), but specifically returns a string.
+  * This is a convenience function for working with the Closure Compiler.
+  * @param {string} name The name of the option (e.g. 'strokeWidth')
+  * @param {string=} opt_seriesName Series name to get per-series values.
+  * @return {string} The value of the option.
+  * @private
+  */
+ Dygraph.prototype.getStringOption = function(name, opt_seriesName) {
+   return /** @type{string} */(this.getOption(name, opt_seriesName));
+ };
+ /**
+  * Like getOption(), but specifically returns a boolean.
+  * This is a convenience function for working with the Closure Compiler.
+  * @param {string} name The name of the option (e.g. 'strokeWidth')
+  * @param {string=} opt_seriesName Series name to get per-series values.
+  * @return {boolean} The value of the option.
+  * @private
+  */
+ Dygraph.prototype.getBooleanOption = function(name, opt_seriesName) {
+   return /** @type{boolean} */(this.getOption(name, opt_seriesName));
+ };
+ /**
+  * Like getOption(), but specifically returns a function.
+  * This is a convenience function for working with the Closure Compiler.
+  * @param {string} name The name of the option (e.g. 'strokeWidth')
+  * @param {string=} opt_seriesName Series name to get per-series values.
+  * @return {function(...)} The value of the option.
+  * @private
+  */
+ Dygraph.prototype.getFunctionOption = function(name, opt_seriesName) {
+   return /** @type{function(...)} */(this.getOption(name, opt_seriesName));
+ };
  Dygraph.prototype.getOptionForAxis = function(name, axis) {
    return this.attributes_.getForAxis(name, axis);
  };
  
  /**
   * @private
-  * @param  String} axis The name of the axis (i.e. 'x', 'y' or 'y2')
+  * @param {string} axis The name of the axis (i.e. 'x', 'y' or 'y2')
   * @return { ... } A function mapping string -> option value
   */
  Dygraph.prototype.optionsViewForAxis_ = function(axis) {
      if (axis_opts && axis_opts[axis] && axis_opts[axis].hasOwnProperty(opt)) {
        return axis_opts[axis][opt];
      }
 +
 +    // I don't like that this is in a second spot.
 +    if (axis === 'x' && opt === 'logscale') {
 +      // return the default value.
 +      // TODO(konigsberg): pull the default from a global default.
 +      return false;
 +    }
 +
      // user-specified attributes always trump defaults, even if they're less
      // specific.
      if (typeof(self.user_attrs_[opt]) != 'undefined') {
  
  /**
   * Returns the current rolling period, as set by the user or an option.
-  * @return {Number} The number of points in the rolling window
+  * @return {number} The number of points in the rolling window
   */
  Dygraph.prototype.rollPeriod = function() {
    return this.rollPeriod_;
@@@ -736,7 -778,7 +786,7 @@@ Dygraph.prototype.xAxisRange = function
   * data set.
   */
  Dygraph.prototype.xAxisExtremes = function() {
-   var pad = this.attr_('xRangePad') / this.plotter_.area.w;
+   var pad = this.getNumericOption('xRangePad') / this.plotter_.area.w;
    if (this.numRows() === 0) {
      return [0 - pad, 1 + pad];
    }
@@@ -850,37 -892,7 +900,37 @@@ Dygraph.prototype.toDataXCoord = functi
  
    var area = this.plotter_.area;
    var xRange = this.xAxisRange();
 -  return xRange[0] + (x - area.x) / area.w * (xRange[1] - xRange[0]);
 +
 +  if (!this.attributes_.getForAxis("logscale", 'x')) {
 +    return xRange[0] + (x - area.x) / area.w * (xRange[1] - xRange[0]);
 +  } else {
 +    // TODO: remove duplicate code?
 +    // Computing the inverse of toDomCoord.
 +    var pct = (x - area.x) / area.w;
 +
 +    // Computing the inverse of toPercentXCoord. The function was arrived at with
 +    // the following steps:
 +    //
 +    // Original calcuation:
 +    // pct = (log(x) - log(xRange[0])) / (log(xRange[1]) - log(xRange[0])));
 +    //
 +    // Multiply both sides by the right-side demoninator.
 +    // pct * (log(xRange[1] - log(xRange[0]))) = log(x) - log(xRange[0])
 +    //
 +    // add log(xRange[0]) to both sides
 +    // log(xRange[0]) + (pct * (log(xRange[1]) - log(xRange[0])) = log(x);
 +    //
 +    // Swap both sides of the equation,
 +    // log(x) = log(xRange[0]) + (pct * (log(xRange[1]) - log(xRange[0]))
 +    //
 +    // Use both sides as the exponent in 10^exp and we're done.
 +    // x = 10 ^ (log(xRange[0]) + (pct * (log(xRange[1]) - log(xRange[0])))
 +    var logr0 = Dygraph.log10(xRange[0]);
 +    var logr1 = Dygraph.log10(xRange[1]);
 +    var exponent = logr0 + (pct * (logr1 - logr0));
 +    var value = Math.pow(Dygraph.LOG_SCALE, exponent);
 +    return value;
 +  }
  };
  
  /**
@@@ -908,25 -920,21 +958,25 @@@ Dygraph.prototype.toDataYCoord = functi
      // the following steps:
      //
      // Original calcuation:
 -    // pct = (logr1 - Dygraph.log10(y)) / (logr1 - Dygraph.log10(yRange[0]));
 +    // pct = (log(yRange[1]) - log(y)) / (log(yRange[1]) - log(yRange[0]));
      //
 -    // Move denominator to both sides:
 -    // pct * (logr1 - Dygraph.log10(yRange[0])) = logr1 - Dygraph.log10(y);
 +    // Multiply both sides by the right-side demoninator.
 +    // pct * (log(yRange[1]) - log(yRange[0])) = log(yRange[1]) - log(y);
      //
 -    // subtract logr1, and take the negative value.
 -    // logr1 - (pct * (logr1 - Dygraph.log10(yRange[0]))) = Dygraph.log10(y);
 +    // subtract log(yRange[1]) from both sides.
 +    // (pct * (log(yRange[1]) - log(yRange[0]))) - log(yRange[1]) = -log(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])));
 -
 +    // and multiply both sides by -1.
 +    // log(yRange[1]) - (pct * (logr1 - log(yRange[0])) = log(y);
 +    //
 +    // Swap both sides of the equation,
 +    // log(y) = log(yRange[1]) - (pct * (log(yRange[1]) - log(yRange[0])));
 +    //
 +    // Use both sides as the exponent in 10^exp and we're done.
 +    // y = 10 ^ (log(yRange[1]) - (pct * (log(yRange[1]) - log(yRange[0]))));
 +    var logr0 = Dygraph.log10(yRange[0]);
      var logr1 = Dygraph.log10(yRange[1]);
 -    var exponent = logr1 - (pct * (logr1 - Dygraph.log10(yRange[0])));
 +    var exponent = logr1 - (pct * (logr1 - logr0));
      var value = Math.pow(Dygraph.LOG_SCALE, exponent);
      return value;
    }
   * If y is null, this returns null.
   * if axis is null, this uses the first axis.
   *
-  * @param { Number } y The data y-coordinate.
-  * @param { Number } [axis] The axis number on which the data coordinate lives.
-  * @return { Number } A fraction in [0, 1] where 0 = the top edge.
+  * @param {number} y The data y-coordinate.
+  * @param {number} [axis] The axis number on which the data coordinate lives.
+  * @return {number} A fraction in [0, 1] where 0 = the top edge.
   */
  Dygraph.prototype.toPercentYCoord = function(y, axis) {
    if (y === null) {
  
    var pct;
    var logscale = this.attributes_.getForAxis("logscale", axis);
 -  if (!logscale) {
 +  if (logscale) {
 +    var logr0 = Dygraph.log10(yRange[0]);
 +    var logr1 = Dygraph.log10(yRange[1]);
 +    pct = (logr1 - Dygraph.log10(y)) / (logr1 - logr0);
 +  } else {
      // 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;
  };
   * values can fall outside the canvas.
   *
   * If x is null, this returns null.
-  * @param { Number } x The data x-coordinate.
-  * @return { Number } A fraction in [0, 1] where 0 = the left edge.
+  * @param {number} x The data x-coordinate.
+  * @return {number} A fraction in [0, 1] where 0 = the left edge.
   */
  Dygraph.prototype.toPercentXCoord = function(x) {
    if (x === null) {
    }
  
    var xRange = this.xAxisRange();
 -  return (x - xRange[0]) / (xRange[1] - xRange[0]);
 +  var pct;
 +  var logscale = this.attributes_.getForAxis("logscale", 'x') ;
 +  if (logscale == true) { // logscale can be null so we test for true explicitly.
 +    var logr0 = Dygraph.log10(xRange[0]);
 +    var logr1 = Dygraph.log10(xRange[1]);
 +    pct = (Dygraph.log10(x) - logr0) / (logr1 - logr0);
 +  } else {
 +    // x - xRange[0] is unit distance from the left.
 +    // xRange[1] - xRange[0] is the scale of the range.
 +    // The full expression below is the % from the left.
 +    pct = (x - xRange[0]) / (xRange[1] - xRange[0]);
 +  }
 +  return pct;
  };
  
  /**
   * Returns the number of columns (including the independent variable).
-  * @return { Integer } The number of columns.
+  * @return {number} The number of columns.
   */
  Dygraph.prototype.numColumns = function() {
    if (!this.rawData_) return 0;
  
  /**
   * Returns the number of rows (excluding any header/label row).
-  * @return { Integer } The number of rows, less any header.
+  * @return {number} The number of rows, less any header.
   */
  Dygraph.prototype.numRows = function() {
    if (!this.rawData_) return 0;
   * Returns the value in the given row and column. If the row and column exceed
   * the bounds on the data, returns null. Also returns null if the value is
   * missing.
-  * @param { Number} row The row number of the data (0-based). Row 0 is the
-  * first row of data, not a header row.
-  * @param { Number} col The column number of the data (0-based)
-  * @return { Number } The value in the specified cell or null if the row/col
-  * were out of range.
+  * @param {number} row The row number of the data (0-based). Row 0 is the
+  *     first row of data, not a header row.
+  * @param {number} col The column number of the data (0-based)
+  * @return {number} The value in the specified cell or null if the row/col
+  *     were out of range.
   */
  Dygraph.prototype.getValue = function(row, col) {
    if (row < 0 || row > this.rawData_.length) return null;
@@@ -1063,11 -1058,11 +1113,11 @@@ Dygraph.prototype.createInterface_ = fu
    // ... and for static parts of the chart.
    this.hidden_ = this.createPlotKitCanvas_(this.canvas_);
  
-   this.resizeElements_();
    this.canvas_ctx_ = Dygraph.getContext(this.canvas_);
    this.hidden_ctx_ = Dygraph.getContext(this.hidden_);
  
+   this.resizeElements_();
    // The interactive parts of the graph are drawn on top of the chart.
    this.graphDiv.appendChild(this.hidden_);
    this.graphDiv.appendChild(this.canvas_);
  Dygraph.prototype.resizeElements_ = function() {
    this.graphDiv.style.width = this.width_ + "px";
    this.graphDiv.style.height = this.height_ + "px";
-   this.canvas_.width = this.width_;
-   this.canvas_.height = this.height_;
+   var canvasScale = Dygraph.getContextPixelRatio(this.canvas_ctx_);
+   this.canvas_.width = this.width_ * canvasScale;
+   this.canvas_.height = this.height_ * canvasScale;
    this.canvas_.style.width = this.width_ + "px";    // for IE
    this.canvas_.style.height = this.height_ + "px";  // for IE
-   this.hidden_.width = this.width_;
-   this.hidden_.height = this.height_;
+   if (canvasScale !== 1) {
+     this.canvas_ctx_.scale(canvasScale, canvasScale);
+   }
+   var hiddenScale = Dygraph.getContextPixelRatio(this.hidden_ctx_);
+   this.hidden_.width = this.width_ * hiddenScale;
+   this.hidden_.height = this.height_ * hiddenScale;
    this.hidden_.style.width = this.width_ + "px";    // for IE
    this.hidden_.style.height = this.height_ + "px";  // for IE
+   if (hiddenScale !== 1) {
+     this.hidden_ctx_.scale(hiddenScale, hiddenScale);
+   }
  };
  
  /**
@@@ -1221,11 -1226,11 +1281,11 @@@ Dygraph.prototype.setColors_ = function
    this.colorsMap_ = {};
  
    // These are used for when no custom colors are specified.
-   var sat = this.attr_('colorSaturation') || 1.0;
-   var val = this.attr_('colorValue') || 0.5;
+   var sat = this.getNumericOption('colorSaturation') || 1.0;
+   var val = this.getNumericOption('colorValue') || 0.5;
    var half = Math.ceil(num / 2);
  
-   var colors = this.attr_('colors');
+   var colors = this.getOption('colors');
    var visibility = this.visibility();
    for (var i = 0; i < num; i++) {
      if (!visibility[i]) {
   * Return the list of colors. This is either the list of colors passed in the
   * attributes or the autogenerated list of rgb(r,g,b) strings.
   * This does not return colors for invisible series.
-  * @return {Array<string>} The list of colors.
+  * @return {Array.<string>} The list of colors.
   */
  Dygraph.prototype.getColors = function() {
    return this.colors_;
@@@ -1300,7 -1305,7 +1360,7 @@@ Dygraph.prototype.createRollInterface_ 
      this.graphDiv.appendChild(this.roller_);
    }
  
-   var display = this.attr_('showRoller') ? 'block' : 'none';
+   var display = this.getBooleanOption('showRoller') ? 'block' : 'none';
  
    var area = this.plotter_.area;
    var textAttr = { "position": "absolute",
  };
  
  /**
-  * @private
-  * Converts page the x-coordinate of the event to pixel x-coordinates on the
-  * canvas (i.e. DOM Coords).
-  */
- Dygraph.prototype.dragGetX_ = function(e, context) {
-   return Dygraph.pageX(e) - context.px;
- };
- /**
-  * @private
-  * Converts page the y-coordinate of the event to pixel y-coordinates on the
-  * canvas (i.e. DOM Coords).
-  */
- Dygraph.prototype.dragGetY_ = function(e, context) {
-   return Dygraph.pageY(e) - context.py;
- };
- /**
   * Set up all the mouse handlers needed to capture dragging behavior for zoom
   * events.
   * @private
@@@ -1396,16 -1383,17 +1438,17 @@@ Dygraph.prototype.createDragInterface_ 
          event.cancelBubble = true;
        }
  
-       contextB.px = Dygraph.findPosX(g.canvas_);
-       contextB.py = Dygraph.findPosY(g.canvas_);
-       contextB.dragStartX = g.dragGetX_(event, contextB);
-       contextB.dragStartY = g.dragGetY_(event, contextB);
+       var canvasPos = Dygraph.findPos(g.canvas_);
+       contextB.px = canvasPos.x;
+       contextB.py = canvasPos.y;
+       contextB.dragStartX = Dygraph.dragGetX_(event, contextB);
+       contextB.dragStartY = Dygraph.dragGetY_(event, contextB);
        contextB.cancelNextDblclick = false;
        contextB.tarp.cover();
      }
    };
  
-   var interactionModel = this.attr_("interactionModel");
+   var interactionModel = this.getOption("interactionModel");
  
    // Self is the graph.
    var self = this;
   * avoid extra redrawing, but it's tricky to avoid interactions with the status
   * dots.
   *
-  * @param {Number} direction the direction of the zoom rectangle. Acceptable
-  * values are Dygraph.HORIZONTAL and Dygraph.VERTICAL.
-  * @param {Number} startX The X position where the drag started, in canvas
-  * coordinates.
-  * @param {Number} endX The current X position of the drag, in canvas coords.
-  * @param {Number} startY The Y position where the drag started, in canvas
-  * coordinates.
-  * @param {Number} endY The current Y position of the drag, in canvas coords.
-  * @param {Number} prevDirection the value of direction on the previous call to
-  * this function. Used to avoid excess redrawing
-  * @param {Number} prevEndX The value of endX on the previous call to this
-  * function. Used to avoid excess redrawing
-  * @param {Number} prevEndY The value of endY on the previous call to this
-  * function. Used to avoid excess redrawing
+  * @param {number} direction the direction of the zoom rectangle. Acceptable
+  *     values are Dygraph.HORIZONTAL and Dygraph.VERTICAL.
+  * @param {number} startX The X position where the drag started, in canvas
+  *     coordinates.
+  * @param {number} endX The current X position of the drag, in canvas coords.
+  * @param {number} startY The Y position where the drag started, in canvas
+  *     coordinates.
+  * @param {number} endY The current Y position of the drag, in canvas coords.
+  * @param {number} prevDirection the value of direction on the previous call to
+  *     this function. Used to avoid excess redrawing
+  * @param {number} prevEndX The value of endX on the previous call to this
+  *     function. Used to avoid excess redrawing
+  * @param {number} prevEndY The value of endY on the previous call to this
+  *     function. Used to avoid excess redrawing
   * @private
   */
  Dygraph.prototype.drawZoomRect_ = function(direction, startX, endX, startY,
@@@ -1519,8 -1507,8 +1562,8 @@@ Dygraph.prototype.clearZoomRect_ = func
   * points near lowX or highX. Don't confuse this function with doZoomXDates,
   * which accepts dates that match the raw data. This function redraws the graph.
   *
-  * @param {Number} lowX The leftmost pixel value that should be visible.
-  * @param {Number} highX The rightmost pixel value that should be visible.
+  * @param {number} lowX The leftmost pixel value that should be visible.
+  * @param {number} highX The rightmost pixel value that should be visible.
   * @private
   */
  Dygraph.prototype.doZoomX_ = function(lowX, highX) {
  };
  
  /**
 - * Transition function to use in animations. Returns values between 0.0
 - * (totally old values) and 1.0 (totally new values) for each frame.
 - * @private
 - */
 -Dygraph.zoomAnimationFunction = function(frame, numFrames) {
 -  var k = 1.5;
 -  return (1.0 - Math.pow(k, -frame)) / (1.0 - Math.pow(k, -numFrames));
 -};
 -
 -/**
   * Zoom to something containing [minDate, maxDate] values. Don't confuse this
   * method with doZoomX which accepts pixel coordinates. This function redraws
   * the graph.
   *
-  * @param {Number} minDate The minimum date that should be visible.
-  * @param {Number} maxDate The maximum date that should be visible.
+  * @param {number} minDate The minimum date that should be visible.
+  * @param {number} maxDate The maximum date that should be visible.
   * @private
   */
  Dygraph.prototype.doZoomXDates_ = function(minDate, maxDate) {
 -  // TODO(danvk): when yAxisRange is null (i.e. "fit to data", the animation
 -  // can produce strange effects. Rather than the y-axis transitioning slowly
 +  // TODO(danvk): when xAxisRange is null (i.e. "fit to data", the animation
 +  // can produce strange effects. Rather than the x-axis transitioning slowly
    // between values, it can jerk around.)
    var old_window = this.xAxisRange();
    var new_window = [minDate, maxDate];
    this.zoomed_x_ = true;
    var that = this;
    this.doAnimatedZoom(old_window, new_window, null, null, function() {
-     if (that.attr_("zoomCallback")) {
-       that.attr_("zoomCallback")(minDate, maxDate, that.yAxisRanges());
+     if (that.getFunctionOption("zoomCallback")) {
+       that.getFunctionOption("zoomCallback")(
+           minDate, maxDate, that.yAxisRanges());
      }
    });
  };
   * Zoom to something containing [lowY, highY]. These are pixel coordinates in
   * the canvas. This function redraws the graph.
   *
-  * @param {Number} lowY The topmost pixel value that should be visible.
-  * @param {Number} highY The lowest pixel value that should be visible.
+  * @param {number} lowY The topmost pixel value that should be visible.
+  * @param {number} highY The lowest pixel value that should be visible.
   * @private
   */
  Dygraph.prototype.doZoomY_ = function(lowY, highY) {
    this.zoomed_y_ = true;
    var that = this;
    this.doAnimatedZoom(null, null, oldValueRanges, newValueRanges, function() {
-     if (that.attr_("zoomCallback")) {
+     if (that.getFunctionOption("zoomCallback")) {
        var xRange = that.xAxisRange();
-       that.attr_("zoomCallback")(xRange[0], xRange[1], that.yAxisRanges());
+       that.getFunctionOption("zoomCallback")(
+           xRange[0], xRange[1], that.yAxisRanges());
      }
    });
  };
  
  /**
 + * Transition function to use in animations. Returns values between 0.0
 + * (totally old values) and 1.0 (totally new values) for each frame.
 + * @private
 + */
 +Dygraph.zoomAnimationFunction = function(frame, numFrames) {
 +  var k = 1.5;
 +  return (1.0 - Math.pow(k, -frame)) / (1.0 - Math.pow(k, -numFrames));
 +};
 +
 +/**
   * Reset the zoom to the original view coordinates. This is the same as
   * double-clicking on the graph.
   */
@@@ -1628,7 -1618,7 +1673,7 @@@ Dygraph.prototype.resetZoom = function(
  
      // With only one frame, don't bother calculating extreme ranges.
      // TODO(danvk): merge this block w/ the code below.
-     if (!this.attr_("animatedZooms")) {
+     if (!this.getBooleanOption("animatedZooms")) {
        this.dateWindow_ = null;
        for (i = 0; i < this.axes_.length; i++) {
          if (this.axes_[i].valueWindow !== null) {
          }
        }
        this.drawGraph_();
-       if (this.attr_("zoomCallback")) {
-         this.attr_("zoomCallback")(minDate, maxDate, this.yAxisRanges());
+       if (this.getFunctionOption("zoomCallback")) {
+         this.getFunctionOption("zoomCallback")(
+             minDate, maxDate, this.yAxisRanges());
        }
        return;
      }
                delete that.axes_[i].valueWindow;
              }
            }
-           if (that.attr_("zoomCallback")) {
-             that.attr_("zoomCallback")(minDate, maxDate, that.yAxisRanges());
+           if (that.getFunctionOption("zoomCallback")) {
+             that.getFunctionOption("zoomCallback")(
+                 minDate, maxDate, that.yAxisRanges());
            }
          });
    }
   * @private
   */
  Dygraph.prototype.doAnimatedZoom = function(oldXRange, newXRange, oldYRanges, newYRanges, callback) {
-   var steps = this.attr_("animatedZooms") ? Dygraph.ANIMATION_STEPS : 1;
+   var steps = this.getBooleanOption("animatedZooms") ?
+       Dygraph.ANIMATION_STEPS : 1;
  
    var windows = [];
    var valueRanges = [];
@@@ -1750,16 -1743,17 +1798,17 @@@ Dygraph.prototype.eventToDomCoords = fu
    if (event.offsetX && event.offsetY) {
      return [ event.offsetX, event.offsetY ];
    } else {
-     var canvasx = Dygraph.pageX(event) - Dygraph.findPosX(this.mouseEventElement_);
-     var canvasy = Dygraph.pageY(event) - Dygraph.findPosY(this.mouseEventElement_);
+     var eventElementPos = Dygraph.findPos(this.mouseEventElement_);
+     var canvasx = Dygraph.pageX(event) - eventElementPos.x;
+     var canvasy = Dygraph.pageY(event) - eventElementPos.y;
      return [canvasx, canvasy];
    }
  };
  
  /**
   * Given a canvas X coordinate, find the closest row.
-  * @param {Number} domX graph-relative DOM X coordinate
-  * Returns: row number, integer
+  * @param {number} domX graph-relative DOM X coordinate
+  * Returns {number} row number.
   * @private
   */
  Dygraph.prototype.findClosestRow = function(domX) {
   * that's closest to the supplied DOM coordinates using the standard
   * Euclidean X,Y distance.
   *
-  * @param {Number} domX graph-relative DOM X coordinate
-  * @param {Number} domY graph-relative DOM Y coordinate
+  * @param {number} domX graph-relative DOM X coordinate
+  * @param {number} domY graph-relative DOM Y coordinate
   * Returns: {row, seriesName, point}
   * @private
   */
@@@ -1829,8 -1823,8 +1878,8 @@@ Dygraph.prototype.findClosestPoint = fu
   * then finds the series which puts the Y coordinate on top of its filled area,
   * using linear interpolation between adjacent point pairs.
   *
-  * @param {Number} domX graph-relative DOM X coordinate
-  * @param {Number} domY graph-relative DOM Y coordinate
+  * @param {number} domX graph-relative DOM X coordinate
+  * @param {number} domY graph-relative DOM Y coordinate
   * Returns: {row, seriesName, point}
   * @private
   */
@@@ -1896,11 -1890,11 +1945,11 @@@ Dygraph.prototype.mouseMove_ = function
    var canvasx = canvasCoords[0];
    var canvasy = canvasCoords[1];
  
-   var highlightSeriesOpts = this.attr_("highlightSeriesOpts");
+   var highlightSeriesOpts = this.getOption("highlightSeriesOpts");
    var selectionChanged = false;
    if (highlightSeriesOpts && !this.isSeriesLocked()) {
      var closest;
-     if (this.attr_("stackedGraph")) {
+     if (this.getBooleanOption("stackedGraph")) {
        closest = this.findStackedPoint(canvasx, canvasy);
      } else {
        closest = this.findClosestPoint(canvasx, canvasy);
      selectionChanged = this.setSelection(idx);
    }
  
-   var callback = this.attr_("highlightCallback");
+   var callback = this.getFunctionOption("highlightCallback");
    if (callback && selectionChanged) {
      callback(event,
          this.lastx_,
@@@ -1986,9 -1980,9 +2035,9 @@@ Dygraph.prototype.updateSelection_ = fu
    // Clear the previously drawn vertical, if there is one
    var i;
    var ctx = this.canvas_ctx_;
-   if (this.attr_('highlightSeriesOpts')) {
+   if (this.getOption('highlightSeriesOpts')) {
      ctx.clearRect(0, 0, this.width_, this.height_);
-     var alpha = 1.0 - this.attr_('highlightSeriesBackgroundAlpha');
+     var alpha = 1.0 - this.getNumericOption('highlightSeriesBackgroundAlpha');
      if (alpha) {
        // Activating background fade includes an animation effect for a gradual
        // fade. TODO(klausw): make this independently configurable if it causes
      var maxCircleSize = 0;
      var labels = this.attr_('labels');
      for (i = 1; i < labels.length; i++) {
-       var r = this.attr_('highlightCircleSize', labels[i]);
+       var r = this.getNumericOption('highlightCircleSize', labels[i]);
        if (r > maxCircleSize) maxCircleSize = r;
      }
      var px = this.previousVerticalX_;
        var pt = this.selPoints_[i];
        if (!Dygraph.isOK(pt.canvasy)) continue;
  
-       var circleSize = this.attr_('highlightCircleSize', pt.name);
-       var callback = this.attr_("drawHighlightPointCallback", pt.name);
+       var circleSize = this.getNumericOption('highlightCircleSize', pt.name);
+       var callback = this.getFunctionOption("drawHighlightPointCallback", pt.name);
        var color = this.plotter_.colors[pt.name];
        if (!callback) {
          callback = Dygraph.Circles.DEFAULT;
        }
-       ctx.lineWidth = this.attr_('strokeWidth', pt.name);
+       ctx.lineWidth = this.getNumericOption('strokeWidth', pt.name);
        ctx.strokeStyle = color;
        ctx.fillStyle = color;
-       callback(this.g, pt.name, ctx, canvasx, pt.canvasy,
+       callback(this, pt.name, ctx, canvasx, pt.canvasy,
            color, circleSize, pt.idx);
      }
      ctx.restore();
   * Manually set the selected points and display information about them in the
   * legend. The selection can be cleared using clearSelection() and queried
   * using getSelection().
-  * @param { Integer } row number that should be highlighted (i.e. appear with
+  * @param {number} row Row number that should be highlighted (i.e. appear with
   * hover dots on the chart). Set to false to clear any selection.
-  * @param { seriesName } optional series name to highlight that series with the
+  * @param {seriesName} optional series name to highlight that series with the
   * the highlightSeriesOpts setting.
   * @param { locked } optional If true, keep seriesName selected when mousing
   * over the graph, disabling closest-series highlighting. Call clearSelection()
@@@ -2112,11 -2106,11 +2161,11 @@@ Dygraph.prototype.setSelection = functi
   * @private
   */
  Dygraph.prototype.mouseOut_ = function(event) {
-   if (this.attr_("unhighlightCallback")) {
-     this.attr_("unhighlightCallback")(event);
+   if (this.getFunctionOption("unhighlightCallback")) {
+     this.getFunctionOption("unhighlightCallback")(event);
    }
  
-   if (this.attr_("hideOverlayOnMouseOut") && !this.lockedSet_) {
+   if (this.getFunctionOption("hideOverlayOnMouseOut") && !this.lockedSet_) {
      this.clearSelection();
    }
  };
@@@ -2145,7 -2139,7 +2194,7 @@@ Dygraph.prototype.clearSelection = func
  /**
   * Returns the number of the currently selected row. To get data for this row,
   * you can use the getValue method.
-  * @return { Integer } row number, or -1 if nothing is selected
+  * @return {number} row number, or -1 if nothing is selected
   */
  Dygraph.prototype.getSelection = function() {
    if (!this.selPoints_ || this.selPoints_.length < 1) {
@@@ -2181,7 -2175,7 +2230,7 @@@ Dygraph.prototype.isSeriesLocked = func
  
  /**
   * Fires when there's data available to be graphed.
-  * @param {String} data Raw CSV data to be plotted
+  * @param {string} data Raw CSV data to be plotted
   * @private
   */
  Dygraph.prototype.loadedEvent_ = function(data) {
@@@ -2223,14 -2217,14 +2272,14 @@@ Dygraph.prototype.getHandlerClass_ = fu
    if (this.attr_('dataHandler')) {
      handlerClass =  this.attr_('dataHandler');
    } else if (this.fractions_) {
-     if (this.attr_('errorBars')) {
+     if (this.getBooleanOption('errorBars')) {
        handlerClass = Dygraph.DataHandlers.FractionsBarsHandler;
      } else {
        handlerClass = Dygraph.DataHandlers.DefaultFractionHandler;
      }
-   } else if (this.attr_('customBars')) {
+   } else if (this.getBooleanOption('customBars')) {
      handlerClass = Dygraph.DataHandlers.CustomBarsHandler;
-   } else if (this.attr_('errorBars')) {
+   } else if (this.getBooleanOption('errorBars')) {
      handlerClass = Dygraph.DataHandlers.ErrorBarsHandler;
    } else {
      handlerClass = Dygraph.DataHandlers.DefaultHandler;
@@@ -2382,18 -2376,22 +2431,22 @@@ Dygraph.stackPoints_ = function
  
      var actualYval = point.yval;
      if (isNaN(actualYval) || actualYval === null) {
-       // Interpolate/extend for stacking purposes if possible.
-       updateNextPoint(i);
-       if (prevPoint && nextPoint && fillMethod != 'none') {
-         // Use linear interpolation between prevPoint and nextPoint.
-         actualYval = prevPoint.yval + (nextPoint.yval - prevPoint.yval) *
-             ((xval - prevPoint.xval) / (nextPoint.xval - prevPoint.xval));
-       } else if (prevPoint && fillMethod == 'all') {
-         actualYval = prevPoint.yval;
-       } else if (nextPoint && fillMethod == 'all') {
-         actualYval = nextPoint.yval;
-       } else {
+       if(fillMethod == 'none') {
          actualYval = 0;
+       } else {
+         // Interpolate/extend for stacking purposes if possible.
+         updateNextPoint(i);
+         if (prevPoint && nextPoint && fillMethod != 'none') {
+           // Use linear interpolation between prevPoint and nextPoint.
+           actualYval = prevPoint.yval + (nextPoint.yval - prevPoint.yval) *
+               ((xval - prevPoint.xval) / (nextPoint.xval - prevPoint.xval));
+         } else if (prevPoint && fillMethod == 'all') {
+           actualYval = prevPoint.yval;
+         } else if (nextPoint && fillMethod == 'all') {
+           actualYval = nextPoint.yval;
+         } else {
+           actualYval = 0;
+         }
        }
      } else {
        prevPoint = point;
@@@ -2509,14 -2507,14 +2562,14 @@@ Dygraph.prototype.gatherDatasets_ = fun
  
      var seriesName = this.attr_("labels")[seriesIdx];
      var seriesExtremes = this.dataHandler_.getExtremeYValues(series, 
-         dateWindow, this.attr_("stepPlot",seriesName));
+         dateWindow, this.getBooleanOption("stepPlot",seriesName));
  
      var seriesPoints = this.dataHandler_.seriesToPoints(series, 
          seriesName, boundaryIds[seriesIdx-1][0]);
  
-     if (this.attr_("stackedGraph")) {
+     if (this.getBooleanOption("stackedGraph")) {
        Dygraph.stackPoints_(seriesPoints, cumulativeYval, seriesExtremes,
-                            this.attr_("stackedGraphNaNFill"));
+                            this.getBooleanOption("stackedGraphNaNFill"));
      }
  
      extremes[seriesName] = seriesExtremes;
@@@ -2542,7 -2540,7 +2595,7 @@@ Dygraph.prototype.drawGraph_ = function
  
    this.layout_.removeAllDatasets();
    this.setColors_();
-   this.attrs_.pointSize = 0.5 * this.attr_('highlightCircleSize');
+   this.attrs_.pointSize = 0.5 * this.getNumericOption('highlightCircleSize');
  
    var packed = this.gatherDatasets_(this.rolledSeries_, this.dateWindow_);
    var points = packed.points;
    this.layout_.evaluate();
    this.renderGraph_(is_initial_draw);
  
-   if (this.attr_("timingName")) {
+   if (this.getStringOption("timingName")) {
      var end = new Date();
-     Dygraph.info(this.attr_("timingName") + " - drawGraph: " + (end - start) + "ms");
+     Dygraph.info(this.getStringOption("timingName") + " - drawGraph: " + (end - start) + "ms");
    }
  };
  
@@@ -2590,10 -2588,10 +2643,10 @@@ Dygraph.prototype.renderGraph_ = functi
    this.cascadeEvents_('clearChart');
    this.plotter_.clear();
  
-   if (this.attr_('underlayCallback')) {
+   if (this.getFunctionOption('underlayCallback')) {
      // NOTE: we pass the dygraph object to this callback twice to avoid breaking
      // users who expect a deprecated form of this callback.
-     this.attr_('underlayCallback')(
+     this.getFunctionOption('underlayCallback')(
          this.hidden_ctx_, this.layout_.getPlotArea(), this, this);
    }
  
    this.canvas_.getContext('2d').clearRect(0, 0, this.canvas_.width,
                                            this.canvas_.height);
  
-   if (this.attr_("drawCallback") !== null) {
-     this.attr_("drawCallback")(this, is_initial_draw);
+   if (this.getFunctionOption("drawCallback") !== null) {
+     this.getFunctionOption("drawCallback")(this, is_initial_draw);
    }
    if (is_initial_draw) {
      this.readyFired_ = true;
@@@ -2693,7 -2691,7 +2746,7 @@@ Dygraph.prototype.computeYAxes_ = funct
  
  /**
   * Returns the number of y-axes on the chart.
-  * @return {Number} the number of axes.
+  * @return {number} the number of axes.
   */
  Dygraph.prototype.numAxes = function() {
    return this.attributes_.numAxes();
  /**
   * @private
   * Returns axis properties for the given series.
-  * @param { String } setName The name of the series for which to get axis
+  * @param {string} setName The name of the series for which to get axis
   * properties, e.g. 'Y1'.
-  * @return { Object } The axis properties.
+  * @return {Object} The axis properties.
   */
  Dygraph.prototype.axisPropertiesForSeries = function(series) {
    // TODO(danvk): handle errors.
@@@ -2750,10 -2748,10 +2803,10 @@@ Dygraph.prototype.computeYAxisRanges_ 
      //
      ypadCompat = true;
      ypad = 0.1; // add 10%
-     if (this.attr_('yRangePad') !== null) {
+     if (this.getNumericOption('yRangePad') !== null) {
        ypadCompat = false;
        // Convert pixel padding to ratio
-       ypad = this.attr_('yRangePad') / this.plotter_.area.h;
+       ypad = this.getNumericOption('yRangePad') / this.plotter_.area.h;
      }
  
      if (series.length === 0) {
  
          // Backwards-compatible behavior: Move the span to start or end at zero if it's
          // close to zero, but not if avoidMinZero is set.
-         if (ypadCompat && !this.attr_("avoidMinZero")) {
+         if (ypadCompat && !this.getBooleanOption("avoidMinZero")) {
            if (minAxisY < 0 && minY >= 0) minAxisY = 0;
            if (maxAxisY > 0 && maxY <= 0) maxAxisY = 0;
          }
  /**
   * Detects the type of the str (date or numeric) and sets the various
   * formatting attributes in this.attrs_ based on this type.
-  * @param {String} str An x value.
+  * @param {string} str An x value.
   * @private
   */
  Dygraph.prototype.detectTypeFromString_ = function(str) {
@@@ -2929,46 -2927,12 +2982,12 @@@ Dygraph.prototype.setXAxisOptions_ = fu
      // TODO(danvk): use Dygraph.numberValueFormatter here?
      /** @private (shut up, jsdoc!) */
      this.attrs_.axes.x.valueFormatter = function(x) { return x; };
 -    this.attrs_.axes.x.ticker = Dygraph.numericLinearTicks;
 +    this.attrs_.axes.x.ticker = Dygraph.numericTicks;
      this.attrs_.axes.x.axisLabelFormatter = this.attrs_.axes.x.valueFormatter;
    }
  };
  
  /**
-  * Parses the value as a floating point number. This is like the parseFloat()
-  * built-in, but with a few differences:
-  * - the empty string is parsed as null, rather than NaN.
-  * - if the string cannot be parsed at all, an error is logged.
-  * If the string can't be parsed, this method returns null.
-  * @param {String} x The string to be parsed
-  * @param {Number} opt_line_no The line number from which the string comes.
-  * @param {String} opt_line The text of the line from which the string comes.
-  * @private
-  */
- // Parse the x as a float or return null if it's not a number.
- Dygraph.prototype.parseFloat_ = function(x, opt_line_no, opt_line) {
-   var val = parseFloat(x);
-   if (!isNaN(val)) return val;
-   // Try to figure out what happeend.
-   // If the value is the empty string, parse it as null.
-   if (/^ *$/.test(x)) return null;
-   // If it was actually "NaN", return it as NaN.
-   if (/^ *nan *$/i.test(x)) return NaN;
-   // Looks like a parsing error.
-   var msg = "Unable to parse '" + x + "' as a number";
-   if (opt_line !== null && opt_line_no !== null) {
-     msg += " on line " + (1+opt_line_no) + " ('" + opt_line + "') of CSV.";
-   }
-   this.error(msg);
-   return null;
- };
- /**
   * @private
   * Parses a string in a special csv format.  We expect a csv file where each
   * line is a date point, and the first field in each line is the date string.
@@@ -2992,7 -2956,7 +3011,7 @@@ Dygraph.prototype.parseCSV_ = function(
    var vals, j;
  
    // Use the default delimiter or fall back to a tab if that makes sense.
-   var delim = this.attr_('delimiter');
+   var delim = this.getStringOption('delimiter');
    if (lines[0].indexOf(delim) == -1 && lines[0].indexOf('\t') >= 0) {
      delim = '\t';
    }
      var fields = [];
      if (!defaultParserSet) {
        this.detectTypeFromString_(inFields[0]);
-       xParser = this.attr_("xValueParser");
+       xParser = this.getFunctionOption("xValueParser");
        defaultParserSet = true;
      }
      fields[0] = xParser(inFields[0], this);
          // TODO(danvk): figure out an appropriate way to flag parse errors.
          vals = inFields[j].split("/");
          if (vals.length != 2) {
-           this.error('Expected fractional "num/den" values in CSV data ' +
-                      "but found a value '" + inFields[j] + "' on line " +
-                      (1 + i) + " ('" + line + "') which is not of this form.");
+           Dygraph.error('Expected fractional "num/den" values in CSV data ' +
+                         "but found a value '" + inFields[j] + "' on line " +
+                         (1 + i) + " ('" + line + "') which is not of this form.");
            fields[j] = [0, 0];
          } else {
-           fields[j] = [this.parseFloat_(vals[0], i, line),
-                        this.parseFloat_(vals[1], i, line)];
+           fields[j] = [Dygraph.parseFloat_(vals[0], i, line),
+                        Dygraph.parseFloat_(vals[1], i, line)];
          }
        }
-     } else if (this.attr_("errorBars")) {
+     } else if (this.getBooleanOption("errorBars")) {
        // If there are error bars, values are (value, stddev) pairs
        if (inFields.length % 2 != 1) {
-         this.error('Expected alternating (value, stdev.) pairs in CSV data ' +
-                    'but line ' + (1 + i) + ' has an odd number of values (' +
-                    (inFields.length - 1) + "): '" + line + "'");
+         Dygraph.error('Expected alternating (value, stdev.) pairs in CSV data ' +
+                       'but line ' + (1 + i) + ' has an odd number of values (' +
+                       (inFields.length - 1) + "): '" + line + "'");
        }
        for (j = 1; j < inFields.length; j += 2) {
-         fields[(j + 1) / 2] = [this.parseFloat_(inFields[j], i, line),
-                                this.parseFloat_(inFields[j + 1], i, line)];
+         fields[(j + 1) / 2] = [Dygraph.parseFloat_(inFields[j], i, line),
+                                Dygraph.parseFloat_(inFields[j + 1], i, line)];
        }
-     } else if (this.attr_("customBars")) {
+     } else if (this.getBooleanOption("customBars")) {
        // Bars are a low;center;high tuple
        for (j = 1; j < inFields.length; j++) {
          var val = inFields[j];
          } else {
            vals = val.split(";");
            if (vals.length == 3) {
-             fields[j] = [ this.parseFloat_(vals[0], i, line),
-                           this.parseFloat_(vals[1], i, line),
-                           this.parseFloat_(vals[2], i, line) ];
+             fields[j] = [ Dygraph.parseFloat_(vals[0], i, line),
+                           Dygraph.parseFloat_(vals[1], i, line),
+                           Dygraph.parseFloat_(vals[2], i, line) ];
            } else {
-             this.warn('When using customBars, values must be either blank ' +
-                       'or "low;center;high" tuples (got "' + val +
-                       '" on line ' + (1+i));
+             Dygraph.warn('When using customBars, values must be either blank ' +
+                          'or "low;center;high" tuples (got "' + val +
+                          '" on line ' + (1+i));
            }
          }
        }
      } else {
        // Values are just numbers
        for (j = 1; j < inFields.length; j++) {
-         fields[j] = this.parseFloat_(inFields[j], i, line);
+         fields[j] = Dygraph.parseFloat_(inFields[j], i, line);
        }
      }
      if (ret.length > 0 && fields[0] < ret[ret.length - 1][0]) {
      }
  
      if (fields.length != expectedCols) {
-       this.error("Number of columns in line " + i + " (" + fields.length +
-                  ") does not agree with number of labels (" + expectedCols +
-                  ") " + line);
+       Dygraph.error("Number of columns in line " + i + " (" + fields.length +
+                     ") does not agree with number of labels (" + expectedCols +
+                     ") " + line);
      }
  
      // If the user specified the 'labels' option and none of the cells of the
          if (fields[j]) all_null = false;
        }
        if (all_null) {
-         this.warn("The dygraphs 'labels' option is set, but the first row of " +
-                   "CSV data ('" + line + "') appears to also contain labels. " +
-                   "Will drop the CSV labels and use the option labels.");
+         Dygraph.warn("The dygraphs 'labels' option is set, but the first row " +
+                      "of CSV data ('" + line + "') appears to also contain " +
+                      "labels. Will drop the CSV labels and use the option " +
+                      "labels.");
          continue;
        }
      }
    }
  
    if (outOfOrder) {
-     this.warn("CSV is out of order; order it correctly to speed loading.");
+     Dygraph.warn("CSV is out of order; order it correctly to speed loading.");
      ret.sort(function(a,b) { return a[0] - b[0]; });
    }
  
  };
  
  /**
-  * @private
   * The user has provided their data as a pre-packaged JS array. If the x values
   * are numeric, this is the same as dygraphs' internal format. If the x values
   * are dates, we need to convert them from Date objects to ms since epoch.
-  * @param {[Object]} data
-  * @return {[Object]} data with numeric x values.
+  * @param {!Array} data
+  * @return {Object} data with numeric x values.
+  * @private
   */
  Dygraph.prototype.parseArray_ = function(data) {
    // Peek at the first x value to see if it's numeric.
    if (data.length === 0) {
-     this.error("Can't plot empty data set");
+     Dygraph.error("Can't plot empty data set");
      return null;
    }
    if (data[0].length === 0) {
-     this.error("Data set cannot contain an empty row");
+     Dygraph.error("Data set cannot contain an empty row");
      return null;
    }
  
    var i;
    if (this.attr_("labels") === null) {
-     this.warn("Using default labels. Set labels explicitly via 'labels' " +
-               "in the options parameter");
+     Dygraph.warn("Using default labels. Set labels explicitly via 'labels' " +
+                  "in the options parameter");
      this.attrs_.labels = [ "X" ];
      for (i = 1; i < data[0].length; i++) {
        this.attrs_.labels.push("Y" + i); // Not user_attrs_.
    } else {
      var num_labels = this.attr_("labels");
      if (num_labels.length != data[0].length) {
-       this.error("Mismatch between number of labels (" + num_labels +
-           ") and number of columns in array (" + data[0].length + ")");
+       Dygraph.error("Mismatch between number of labels (" + num_labels + ")" +
+                     " and number of columns in array (" + data[0].length + ")");
        return null;
      }
    }
      var parsedData = Dygraph.clone(data);
      for (i = 0; i < data.length; i++) {
        if (parsedData[i].length === 0) {
-         this.error("Row " + (1 + i) + " of data is empty");
+         Dygraph.error("Row " + (1 + i) + " of data is empty");
          return null;
        }
        if (parsedData[i][0] === null ||
            typeof(parsedData[i][0].getTime) != 'function' ||
            isNaN(parsedData[i][0].getTime())) {
-         this.error("x value in row " + (1 + i) + " is not a Date");
+         Dygraph.error("x value in row " + (1 + i) + " is not a Date");
          return null;
        }
        parsedData[i][0] = parsedData[i][0].getTime();
      // Some intelligent defaults for a numeric x-axis.
      /** @private (shut up, jsdoc!) */
      this.attrs_.axes.x.valueFormatter = function(x) { return x; };
 -    this.attrs_.axes.x.ticker = Dygraph.numericLinearTicks;
 +    this.attrs_.axes.x.ticker = Dygraph.numericTicks;
      this.attrs_.axes.x.axisLabelFormatter = Dygraph.numberAxisLabelFormatter;
      return data;
    }
   * number. All subsequent columns must be numbers. If there is a clear mismatch
   * between this.xValueParser_ and the type of the first column, it will be
   * fixed. Fills out rawData_.
-  * @param {[Object]} data See above.
+  * @param {!google.visualization.DataTable} data See above.
   * @private
   */
  Dygraph.prototype.parseDataTable_ = function(data) {
    } else if (indepType == 'number') {
      this.attrs_.xValueParser = function(x) { return parseFloat(x); };
      this.attrs_.axes.x.valueFormatter = function(x) { return x; };
 -    this.attrs_.axes.x.ticker = Dygraph.numericLinearTicks;
 +    this.attrs_.axes.x.ticker = Dygraph.numericTicks;
      this.attrs_.axes.x.axisLabelFormatter = this.attrs_.axes.x.valueFormatter;
    } else {
-     this.error("only 'date', 'datetime' and 'number' types are supported for " +
-                "column 1 of DataTable input (Got '" + indepType + "')");
+     Dygraph.error("only 'date', 'datetime' and 'number' types are supported " +
+                   "for column 1 of DataTable input (Got '" + indepType + "')");
      return null;
    }
  
      var type = data.getColumnType(i);
      if (type == 'number') {
        colIdx.push(i);
-     } else if (type == 'string' && this.attr_('displayAnnotations')) {
+     } else if (type == 'string' && this.getBooleanOption('displayAnnotations')) {
        // This is OK -- it's an annotation column.
        var dataIdx = colIdx[colIdx.length - 1];
        if (!annotationCols.hasOwnProperty(dataIdx)) {
        }
        hasAnnotations = true;
      } else {
-       this.error("Only 'number' is supported as a dependent type with Gviz." +
-                  " 'string' is only supported if displayAnnotations is true");
+       Dygraph.error("Only 'number' is supported as a dependent type with Gviz." +
+                     " 'string' is only supported if displayAnnotations is true");
      }
    }
  
    var labels = [data.getColumnLabel(0)];
    for (i = 0; i < colIdx.length; i++) {
      labels.push(data.getColumnLabel(colIdx[i]));
-     if (this.attr_("errorBars")) i += 1;
+     if (this.getBooleanOption("errorBars")) i += 1;
    }
    this.attrs_.labels = labels;
    cols = labels.length;
      var row = [];
      if (typeof(data.getValue(i, 0)) === 'undefined' ||
          data.getValue(i, 0) === null) {
-       this.warn("Ignoring row " + i +
-                 " of DataTable because of undefined or null first column.");
+       Dygraph.warn("Ignoring row " + i +
+                    " of DataTable because of undefined or null first column.");
        continue;
      }
  
      } else {
        row.push(data.getValue(i, 0));
      }
-     if (!this.attr_("errorBars")) {
+     if (!this.getBooleanOption("errorBars")) {
        for (j = 0; j < colIdx.length; j++) {
          var col = colIdx[j];
          row.push(data.getValue(i, col));
    }
  
    if (outOfOrder) {
-     this.warn("DataTable is out of order; order it correctly to speed loading.");
+     Dygraph.warn("DataTable is out of order; order it correctly to speed loading.");
      ret.sort(function(a,b) { return a[0] - b[0]; });
    }
    this.rawData_ = ret;
@@@ -3375,7 -3340,7 +3395,7 @@@ Dygraph.prototype.start_ = function() 
        req.send(null);
      }
    } else {
-     this.error("Unknown data format: " + (typeof data));
+     Dygraph.error("Unknown data format: " + (typeof data));
    }
  };
  
   * There's a huge variety of options that can be passed to this method. For a
   * full list, see http://dygraphs.com/options.html.
   *
-  * @param {Object} attrs The new properties and values
-  * @param {Boolean} [block_redraw] Usually the chart is redrawn after every
-  * call to updateOptions(). If you know better, you can pass true to explicitly
-  * block the redraw. This can be useful for chaining updateOptions() calls,
-  * avoiding the occasional infinite loop and preventing redraws when it's not
-  * necessary (e.g. when updating a callback).
+  * @param {Object} input_attrs The new properties and values
+  * @param {boolean} block_redraw Usually the chart is redrawn after every
+  *     call to updateOptions(). If you know better, you can pass true to
+  *     explicitly block the redraw. This can be useful for chaining
+  *     updateOptions() calls, avoiding the occasional infinite loop and
+  *     preventing redraws when it's not necessary (e.g. when updating a
+  *     callback).
   */
  Dygraph.prototype.updateOptions = function(input_attrs, block_redraw) {
    if (typeof(block_redraw) == 'undefined') block_redraw = false;
@@@ -3498,8 -3464,8 +3519,8 @@@ Dygraph.mapLegacyOptions_ = function(at
   * This is far more efficient than destroying and re-instantiating a
   * Dygraph, since it doesn't have to reparse the underlying data.
   *
-  * @param {Number} [width] Width (in pixels)
-  * @param {Number} [height] Height (in pixels)
+  * @param {number} width Width (in pixels)
+  * @param {number} height Height (in pixels)
   */
  Dygraph.prototype.resize = function(width, height) {
    if (this.resize_lock) {
    this.resize_lock = true;
  
    if ((width === null) != (height === null)) {
-     this.warn("Dygraph.resize() should be called with zero parameters or " +
-               "two non-NULL parameters. Pretending it was zero.");
+     Dygraph.warn("Dygraph.resize() should be called with zero parameters or " +
+                  "two non-NULL parameters. Pretending it was zero.");
      width = height = null;
    }
  
  /**
   * Adjusts the number of points in the rolling average. Updates the graph to
   * reflect the new averaging period.
-  * @param {Number} length Number of points over which to average the data.
+  * @param {number} length Number of points over which to average the data.
   */
  Dygraph.prototype.adjustRoll = function(length) {
    this.rollPeriod_ = length;
  Dygraph.prototype.visibility = function() {
    // Do lazy-initialization, so that this happens after we know the number of
    // data series.
-   if (!this.attr_("visibility")) {
+   if (!this.getOption("visibility")) {
      this.attrs_.visibility = [];
    }
    // TODO(danvk): it looks like this could go into an infinite loop w/ user_attrs.
-   while (this.attr_("visibility").length < this.numColumns() - 1) {
+   while (this.getOption("visibility").length < this.numColumns() - 1) {
      this.attrs_.visibility.push(true);
    }
-   return this.attr_("visibility");
+   return this.getOption("visibility");
  };
  
  /**
  Dygraph.prototype.setVisibility = function(num, value) {
    var x = this.visibility();
    if (num < 0 || num >= x.length) {
-     this.warn("invalid series number in setVisibility: " + num);
+     Dygraph.warn("invalid series number in setVisibility: " + num);
    } else {
      x[num] = value;
      this.predraw_();
@@@ -3599,9 -3565,9 +3620,9 @@@ Dygraph.prototype.setAnnotations = func
    Dygraph.addAnnotationRule();
    this.annotations_ = ann;
    if (!this.layout_) {
-     this.warn("Tried to setAnnotations before dygraph was ready. " +
-               "Try setting them in a ready() block. See " +
-               "dygraphs.com/tests/annotation.html");
+     Dygraph.warn("Tried to setAnnotations before dygraph was ready. " +
+                  "Try setting them in a ready() block. See " +
+                  "dygraphs.com/tests/annotation.html");
      return;
    }
  
@@@ -3696,5 -3662,5 +3717,5 @@@ Dygraph.addAnnotationRule = function() 
      }
    }
  
-   this.warn("Unable to add default annotation CSS rule; display may be off.");
+   Dygraph.warn("Unable to add default annotation CSS rule; display may be off.");
  };