Merge branch 'master' of github.com:danvk/dygraphs

lightweight dead code elimination

Make seriesToPoints_ and stackPoints_ private. They are being rendered as part of JSDoc, which is wrong, but also, seriesToPoints_ has the closure substring "<?", which jsdoc likes to think is PHP.

Merge pull request #262 from sauter-hq/left-boundry-fix

Left boundary fix

Fixed wrong space usage in if clauses.

BUGFIX: Fixed bug using the idxToRow_ function instead of directly taking the row from the point. Also the idxToRow_ method has been completely removed based on Dans comment, that it is unneeded.

TEST: Added test to verify that the left boundry works now.

BUGFIX: Added "this." to the  boundryIds_ calls

BUGFIX: Fixed Bug created by extracting different rows for each series (to fix the big not connecting lines at edges). Now all calls to the getLeftBoundry method are done with the correct set index resulting in a correct behaviour now.

Merge branch 'master' of https://github.com/danvk/dygraphs

Add comment to resize test case as a follow on to pull request 260.

Add comment to resize test case as a follow on to pull request 260.

Merge pull request #260 from klausw-g/resize-fix-2

Only resize canvas if the size changed.

Only resize canvas if the size changed.

Resizing the canvas erases it, and Dygraphs had skipped the redraw
when it thought it wasn't necessary, leading to blank graphs after
a redundant .resize() call.

Merge pull request #259 from klausw-g/resize-fix

Fix graphs created in not-displayed divs.

Fix graphs created in not-displayed divs.

This distinguishes two cases:

- graphs with explicit height/width in graph options. They should
  be drawn correctly after display, no resize needed.

- graphs without explicit height/width are supposed to inherit
  the size from the surrounding div. They needs a .resize()
  call after display.

Tests are included for both.

Fixes issue 238.

Merge pull request #258 from klausw-g/lint-fixes

Linter fixes.

Linter fixes.

Refactoring to fix stacked graphs with NaNs.

For stacked graphs, draw gaps for just the specific parts of series that are
missing data, and use interpolation to help ensure that the overall graph shape
doesn't get distorted for missing points.

This replaces #139 "Fix NaN handling for stacked graphs" which had
aggressively propagated NaNs to ensure that graphs don't get drawn with
incorrect stacking.

Fixes issue 446 - Improve handling of NaNs in stacked graphs.

Detailed changes:

* Remove layouts_.datasets, instead create point object arrays directly from
  gatherDatasets_(). This was necessary since the pre-stacked datasets
  aren't able to store the additional data needed for handling gaps.

* For stacked graphs, track yval_stacked/y_stacked separately from yval as point
  properties.

* Remove unstackPointAtIndex which is now no longer necessary, since the points
  keep the original yval. This helps ensure that the returned values keep their
  original NaN value and don't expose the interpolated stacked values used for
  drawing gaps.

* Remove evaluateWithError, evaluateLineCharts and Dygraph.seriesToPoints_ now
  handle error bars directly.

* In gatherDatasets_, remove the unconditional copy of rolledSeries, the
  refactoring appears to have fixed the bug related to zooming with
  error bars on. (I could reproduce it in the original code by zooming
  dygraphs/tests/custom-bars.html.) For date windows (horizontal zoom),
  use .splice() instead of manually coping.

Squashed commit of the following:

commit 56efaef6a50c737b99a8e4e6a36e55c83fde237f
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 16:50:16 2013 -0700

    Style fixes as per Robert's requests.

commit 83bb38a071f407560abcca53b7f6e07d6b0ca0e5
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 15:53:59 2013 -0700

    Add stackedGraphNaNFill option to control NaN handling.

    Add a test for this, and add a docstring to the stackPoints_ method.

commit e16e3c381d5cc7bec9896b31dddebc627a941c33
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 15:20:24 2013 -0700

    Revert name back to .addDataset, add docstring.

commit d3e66a60c84656364829e8a5bb83ed0e2b9fb403
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 15:15:19 2013 -0700

    Add more type annotations to make the flow of point data clearer.

    Unfortunately it's still not very clear, the logic is a bit convoluted.

commit 33db3d27501c8b5f2f84baa4dd92bf8d6f0a35a3
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 15:00:38 2013 -0700

    Rename fixPathAttrs_ to cleanPathAttrs_ as requested.

commit f39168785efcbacd1de9a6275aa02c8e34e4c5a7
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 14:58:26 2013 -0700

    Add comment explaining baseline, as requested by Robert.

commit 80348ed3c9e677fb5ff1ef393056e1e2bea1bbf7
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 13:16:51 2013 -0700

    Track stroke/fill in CanvasAssertions.

    For lineTo operations, set strokeStyle to undefined if the path is
    filled (not stroked), and set fillStyle to undefined if the path
    is stroked only.

    Fix assertions in missing_points tests, those were apparently expecting the
    inflated numbers.

    Remove the strokeStyle='#000000' hack in dygraph-canvas which is now
    no longer needed.

commit 4f47af82ac297d27dc0ac9d5c6fd256e00cccec5
Author: Klaus Weidner <klausw@google.com>
Date:   Mon Jun 3 11:57:56 2013 -0700

    fix CanvasAssertions which was calling .match wrong, remove now-redundant check.

commit 8c53e9e0f3d362e58204ff5a777c90ddb3b589d5
Author: Klaus Weidner <klausw@google.com>
Date:   Thu May 23 11:00:50 2013 -0700

    Restore hasOwnProperty.

commit 621cc37cfaf33821d04de2a94aea63021f7a9647
Author: Klaus Weidner <klausw@google.com>
Date:   Wed May 22 15:35:16 2013 -0700

    Fix all-NaN series stacking and Proxy return values.

    Ensure that all-NaN series get treated as zero for stacking
    purposes, and that nextPoint gets set to null past the last
    stackable point. We want interpolation, but not extrapolation.

    The Proxy class didn't return values from calls, breaking
    testCorrectColors since it didn't get pixel data as expected.

    Update the input data for testInterpolation to check corner cases,
    including an all-NaN series.

commit eb52ff1c1bbdd55312cb2881b5182731c91c155c
Author: Klaus Weidner <klausw@google.com>
Date:   Wed May 22 12:40:44 2013 -0700

    Refactoring to fix stacked graphs with NaNs.

    For stacked graphs, draw gaps for just the specific parts of series that are
    missing data, and use interpolation to help ensure that the overall graph shape
    doesn't get distorted for missing points.

    This replaces https://github.com/danvk/dygraphs/pull/139 "Fix NaN handling for
    stacked graphs" which had aggressively propagated NaNs to ensure that graphs
    don't get drawn with incorrect stacking.

    Fixes issue 446 - Improve handling of NaNs in stacked graphs.

    Detailed changes:

    - Remove layouts_.datasets, instead create point object arrays directly from
      gatherDatasets_(). This was necessary since the pre-stacked datasets aren't
      able to store the additional data needed for handling gaps.

    - For stacked graphs, track yval_stacked/y_stacked separately from yval as point
      properties.

    - Remove unstackPointAtIndex which is now no longer necessary, since the points
      keep the original yval. This helps ensure that the returned values keep their
      original NaN value and don't expose the interpolated stacked values used for
      drawing gaps.

    - Remove evaluateWithError, evaluateLineCharts and Dygraph.seriesToPoints_ now
      handle error bars directly.

    - In gatherDatasets_, remove the unconditional copy of rolledSeries, the
      refactoring appears to have fixed the bug related to zooming with
      error bars on. (I could reproduce it in the original code by zooming
      dygraphs/tests/custom-bars.html.) For date windows (horizontal zoom),
      use .splice() instead of manually coping.

Merge pull request #257 from kberg/auto-tests

Auto tests cleanup and enhancements

Merge branch 'master' of https://github.com/danvk/dygraphs into auto-tests

Merge pull request #250 from wimme/patch-4

LogScale and customBars with negative values

Clean up more of the local testing script, and also make it prettier. Add execution time to header.

Reduce the width of testGridLinePattern from 480px to 120px, making the test run in 150ms instead of ~8000ms.

Merge branch 'master' of https://github.com/danvk/dygraphs

Merge pull request #256 from kberg/auto-tests

I broke test.sh. This fixes it.

Merge branch 'auto-tests' of https://github.com/kberg/dygraphs into auto-tests

I broke ./test.sh, and this fixes it.

I broke ./test.sh, and this fixes it.

Merge pull request #255 from kberg/master

Add --verbose flag to test.sh and phantom-driver.js which gathers all output from the web page under test.

Merge branch 'master' of https://github.com/danvk/dygraphs

Add --verbose to ./test.sh and phantom-driver.js

Merge pull request #254 from kberg/i382

Fix lint errors for i382. Also, ugh, clearly broken bit with mouseUpHandler.

Fix lint errors for i382. Also, ugh, clearly broken bit with mouseUpHandler.

Merge pull request #232 from kberg/i382

Fix bug 382. Simplify resizing by not recreating everything from scratch...

Merge branch 'i382' of https://github.com/kberg/dygraphs into i382

Fix tests that accompany fixing bug 382.

Merge branch 'master' of https://github.com/danvk/dygraphs into i382

Merge pull request #252 from kberg/auto-tests

Add nice little things to the in-browser test runner.

Merge branch 'master' of https://github.com/danvk/dygraphs into i382

Add nice little things to the in-browser test runner.
1. Add meta-links to the test name and new "more..." links.
2. Test name link runs that single test. more... shows the stack trace in an alert window. (Not ideal, but it's something.)

Merge pull request #251 from kberg/auto-tests

Nicer auto tests

Make a table view out of the test results for easier reading.

Rewrite local testdriver processing. Using a listener framework to report test results, and also, store test execution durations.

Turn auto-tests script into an object for saner manipulation.

The local auto tester's javascript is messy and having it embedded in HTML doesn't help, so this change moves it out into its own file.

Update dygraph.js

Update dygraph.js

Add comment

Add test for custom bars with negative values in logScale

LogScale and customBars with negative values

When LogScale and customBars are enabled, it was verifying if the value was negative by using a <= on an array and then inserting a null. This caused exceptions in rollingAverage(), the value can't be null when using customBars.

(This a new patch based on the current code, since pull request #236 was getting old).

Remove the vestigial DygraphLayout.setDateWindow.
(Also, add a clarifying comment.)

Merge branch 'master' of github.com:danvk/dygraphs

Fix failing test

Merge pull request #241 from sauter-hq/per-axis-grid

per-axis grid options

Merge remote-tracking branch 'official/master' into per-axis-grid

Conflicts:
tests/two-axes.html

Merge pull request #243 from sauter-hq/fix-for-issue-451

Fix for issue 451

REFACTORING: fixed code style and lint warnings. Readded jsdoc lost in merge.

REFACTORING: Fixed code style nits and extracted the null values check to a method.

Merge commit '9f890c23ad80924d0a30f3a14f8680b7c2d6318e' into fix-for-issue-451

Conflicts:
dygraph-layout.js

only reserve space for y2 axis when it will be drawn

REFACTORING: cleaned up code and style for new grid per axis feature.

Merge commit 'de2545148870a1bdb0957c4c42e80bdb8ce1656d' into per-axis-grid

Fix our lone lint error

Fix syntax errors in docs and update test script to check for them.

Stack Overflow link on home page

Merge pull request #246 from davidmsibley/master

Quarterly granularity ticker fix

Merge pull request #245 from sauter-hq/custom-bar-data-gaps-459

Custom bar data gaps 459

Merge pull request #237 from kberg/closure

Fix closure bugs in Dygraphs

Follow-on to pull request 237, closurifying some code.

BUGFIX: Fixed test not running die to OS differences. Now the edge pixels arn't checked anymore which works fine on linux and windows.

TEST: Added NaN cases to the missingPoints test.

REFACTORING: Changed if / else cases to optimize the performance for the
default cases.

adding comments for LONG_TICK_PLACEMENTS

Merge branch 'master' of github.com:danvk/dygraphs

Merge pull request #242 from sauter-hq/draw-xaxis-bugfix

BUGFIX: RangeSelector: xAxisLabelHeight is set to 0 in case the option 'drawXAxis' is set to false.

tying all the large granularity tick decision making together

corrected number of months in a QUARTERLY granularity

TEST: Added automatet tests for customBar and errorBar options and missing data points.

BUGFIX: added missing spaces.

Reponses to code review for https://github.com/danvk/dygraphs/pull/237

Merge branch 'master' of github.com:danvk/dygraphs into closure

Conflicts:
dygraph-options.js

BUGFIX: Fixed typo

TEST: testGridLinePattern: Fixed bug not correctly resetting the x-value
in each loop

Merge branch 'per-axis-grid' of https://github.com/sauter-hq/dygraphs into per-axis-grid

REFACTORING: Fixed wrong indents, missing spaces and too much spaces :-)

BUGFIX: Fixed jittering test results by ignoring the alpha value in the color checks.

REFACTORING: fixed jslint violation

REFACTORING: adapted EOL

EXAMPLE: added new example for fixed bug

FEATURE: extended test for the bugfix for issue #451

BUGFIX: bugfix for issue #451 is now also working for bars

TEST: Added examples for the bugfix.

REFACTORING: moved option retrieving out of the for loop for better
performance.

TEST: test to verify the bugfix

BUGFIX: First try of fixing missing data bug for customBars and errorBars

TEST: Added example of range selector without the chart.

TEST: Added automated test proving the bugfix made.

REFACTORING: use the method "getLeftBoundary" instead of a private
variable

BUG FIX: Changed check to undefined with === as proposed by jslint.