can draw and interact!

making progress on a usable dygraph-combined.js; the great quoting begins

file size experiments; possible 36% savings

start using export symbols

really add gviz-api

99 warnings, all in dygraph.js

add gviz-api externs; 99 warnings

114 warnings

a mere 130 warnings

228 warnings in dygraph.js

1 error, 172 warnings on dygraph.js

update command

closurize dygraph-interaction-model.js

stricter types

basic closurization of dygraph-canvas.js

Merge branch 'shrink' into full-closure

Merge branch 'master' into full-closure

remove private method from test

closurize dygraph-layout.js

rv combined

kberg review

export fewer symbols from dygraph.js

notes

notes

test to track exported symbols in dygraph-combined.js

update notes

stop exporting _calcYNormal

consolidate parseFloat_ (!)

export fewer symbols from DygraphCanvasRenderer

export fewer methods

consolidate to static logging functions; hide more symbols

Some cleanup of symbols that should not be exported

silence compressor warnings

switch to uglifyjs for minification; another ~5k savings

consolidate date processing a bit

update notes

Move custom circles into extras/, save some space

Consolidate findPos{X,Y} -> findPos

auto_tests/misc/local works in FF again (which has no innerText, only innerHTML)

test for Dygraph.toRGB_

remove dependence on rgbcolor; unclear if this works in IE

update notes

remove strftime-min.js

remove some dead code

strip supporting code for stacktrace from combined

remove stacktrace.js from combined

track size reducing ideas

CatCompressed -> CatMinified in generate-combined.sh

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.)