Include thousands-separators in `bencher` output

bheisler / criterion.rs

Statistics-driven benchmarking library for Rust

Apache License 2.0

4.31k stars 292 forks source link

Include thousands-separators in `bencher` output #705

Closed d-e-s-o closed 11 months ago

d-e-s-o commented 1 year ago

As mentioned in #704, the output generated when using the bencher output format does not quite match up with that of libtest. One reason being that thousands-separators are not emitted. With this change we adjust the formatting logic for integers (which is only used in bencher style reports) to emit thousands-separators, as libtest does.

danielocfb commented 11 months ago

Thanks for merging. Is there any chance you can cut a new release that includes this change, @lemmih (or others)?