Benchmakr charts Y-axis values offset about benchmarking HOT 6 CLOSED

I've been looking into this, and as an aside point, the instructions about installing Chart.js and Chart.Scatter.js (https://github.com/nodejs/benchmarking/tree/master/tools/chartGen/lib) would benefit by being more specific on versions needed. I found that the lastest versions of the libraries weren't compatible with the chart get code.

@mhdawson was able to provide me a copy of the versions running in CI (Chart.js is at 1.0.2, latest is 2.4.0) and a copy of the results database. My findings so far is that the html that's generated looks correct when viewed in a browser. The problem seems to occur when phantomjs is used to create the png snapshot. This rendering misaligns the data against the Y axis. I'm trying to determine what the cause of this is.

from benchmarking.

The problem was in because the data on the charts animates into view. There was already a 1 second delay for the page to complete before the screen shot was taken, but this wasn't enough to finish the animation. I've just added a PR to turn the animation off - I've tested this and the generated png files look the same as the actual html

from benchmarking.

PR for fix went in, had validated one chart so expect it will resolve. Will validate/close this issue once charts are next fully transferred to benchmarking.nodejs.org.

from benchmarking.

@CurryKitten can you also submit a PR to update the instructions to specify the versions needed so that we can recreate in case of machine loss.

from benchmarking.

@mhdawson Yes - I'll double check those versions and add an appropriate PR

from benchmarking.

Ok, sniff checked acemair latency and value in job from last night matches what we get on the chart in respect to the Y axis in chart on benchmarking.nodejs.org. closing.

from benchmarking.