It's a bit cryptic. It's fine as an implementation, but giving it a name
would make it read better:

var isNullUndefinedOrNaN = function(x) {
  return isNaN(parseFloat(num));
};

  - Dan


On Thu, Jan 31, 2013 at 11:27 AM, Emmanuel Fitoussi <
notifications@github.com> wrote:

> In dygraph.js:
>
> > @@ -2561,7 +2564,10 @@ Dygraph.prototype.computeYAxisRanges_ = function(extremes) {
> >        axis.computedValueRange = [axis.valueWindow[0], axis.valueWindow[1]];
> >      } else if (axis.valueRange) {
> >        // This is a user-set value range for this axis.
> > -      axis.computedValueRange = [axis.valueRange[0], axis.valueRange[1]];
> > +      axis.computedValueRange = [
> > +         !isNaN(axis.valueRange[0]) && axis.valueRange[0] !== null && axis.valueRange[0] !== undefined ? axis.valueRange[0] : axis.extremeRange[0],
>
> I propose this :
>
> isNaN(parseFloat(num))
>
> What do you think about this?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/danvk/dygraphs/pull/199/files#r2845966>.
>
>