benchmark: keep decimals in results

mscdex · mscdex · commit 7889416b8a93 · 2017-01-05T02:48:07.000-05:00
Some benchmarks' results are small values, so keeping decimals when running them manually (not comparing) can be helpful. PR-URL: nodejs#10559 Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/common.js b/benchmark/common.js
@@ -195,8 +195,9 @@ function formatResult(data) {
     conf += ' ' + key + '=' + JSON.stringify(data.conf[key]);
   }
 
-  const rate = Math.floor(data.rate)
-                   .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+  var rate = data.rate.toString().split('.');
+  rate[0] = rate[0].replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+  rate = (rate[1] ? rate.join('.') : rate[0]);
   return `${data.name}${conf}: ${rate}`;
 }
 
diff --git a/benchmark/run.js b/benchmark/run.js
@@ -56,8 +56,9 @@ if (format === 'csv') {
       conf = conf.replace(/"/g, '""');
       console.log(`"${data.name}", "${conf}", ${data.rate}, ${data.time}`);
     } else {
-      const rate = Math.floor(data.rate)
-                       .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+      var rate = data.rate.toString().split('.');
+      rate[0] = rate[0].replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');
+      rate = (rate[1] ? rate.join('.') : rate[0]);
       console.log(`${data.name} ${conf}: ${rate}`);
     }
   });

Original file line number	Diff line number	Diff line change
`@@ -195,8 +195,9 @@ function formatResult(data) {`
`195`	`195`	`conf += ' ' + key + '=' + JSON.stringify(data.conf[key]);`
`196`	`196`	`}`
`197`	`197`
`198`		`- const rate = Math.floor(data.rate)`
`199`		`- .toString().replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');`
	`198`	`+ var rate = data.rate.toString().split('.');`
	`199`	`+ rate[0] = rate[0].replace(/(\d)(?=(\d\d\d)+(?!\d))/g, '$1,');`
	`200`	`+ rate = (rate[1] ? rate.join('.') : rate[0]);`
`200`	`201`	return `${data.name}${conf}: ${rate}`;
`201`	`202`	`}`
`202`	`203`