benchmarking icon indicating copy to clipboard operation
benchmarking copied to clipboard

Published 20 hours ago • verified publisher icon nodejs

Add UglifyJS benchmark

Open mhart opened this issue 8 years ago • 5 comments
trafficstars

As suggested from https://github.com/nodejs/node/issues/11923#issuecomment-291286419 (where some performance regressions were found in v7.x)

mhart avatar Apr 03 '17 22:04 mhart

ping @nodejs/benchmarking

gibfahn avatar Apr 12 '17 21:04 gibfahn

Still seems like a good idea to me. If no one is working on it, I'd be willing to pick it up.

bnoordhuis avatar Apr 13 '17 10:04 bnoordhuis

@bnoordhuis, there are docs in this repo on how to configure to add a new benchmark to the nightly runs/charts but I don't think you'll have all of the access needed to test it out. Most of the work make be in a script to run the benchmark (depending on how much it takes). Once you have a PR I can help test out in advance of landing.

mhdawson avatar Apr 18 '17 20:04 mhdawson

@bnoordhuis you still working on this ?

mhdawson avatar Jun 19 '17 19:06 mhdawson

I think I may simply have forgotten to PR it. I'll try to do that tomorrow.

bnoordhuis avatar Jun 19 '17 19:06 bnoordhuis