search

benchmark-action / github-action-benchmark

GitHub Action for continuous benchmarking to keep performance
https://benchmark-action.github.io/github-action-benchmark/dev/bench/
MIT License
1.02k stars 152 forks source link

Improve wrapping on GitHub comments for benchmarks with long names #263

Open jviotti opened 2 months ago

jviotti commented 2 months ago

I wonder if there is any way to improve the readability of the tables posted by this action when the benchmark names are too long. Take this one for example. We are finding it very hard to read the numbers given the last column gets wrapped.

Screenshot 2024-09-13 at 11 28 28 AM

Shorter benchmark names surely help, but it also sucks to sacrifice on their readability to make them fit... Maybe it would make sense to not put the benchmark case names on the first columns as code blocks using back ticks?

Or maybe there is a way to completely prevent the last column from wrapping? I much rather have the first column wrap.

jviotti commented 2 months ago

Another option might be not include so much precision on the current/previous columns? For example, there is probably no difference between showing 151.74767917302637 and 151.7 us and that could make some significant space?

ktrz commented 2 months ago

Thank you @jviotti for submitting this issue! I'll have a look at what options are available to make this more readable. We could probably get some additional space back by moving the units to a separate column so that ns/iter only occurs once.