-
Couldn't load subscription status.
- Fork 86
Description
The new stats features by #127 introduced two changes in the output, that might not have been intended.
I marked them in the screenshot below, and will try my best to describe them in text as well.
Label of 2nd column
Previous the label of the 2nd column of the stats table was "Value".
Now it is "Average".
However "Average" is not correct for rows 4-6 in the table, as those are just counts.
I don't have a great suggestion here. One could split up the table and introduce a new header for row 4-6.
Not sure how future-proof that would be either though.
Milliseconds in average/median/90th-percentile
The stats in the first 3 rows contain the milliseconds.
They reduce readability a bit, and I doubt that many users will need them.
My recommendation would be to remove the milliseconds.
One might even argue that for most purposes the seconds would not be needed either.
That might be a matter
Backwards compatibility?
@zkoppert had mentioned that introducing a new configuration option for the new stats would be good.
I guess that as the 2nd row still contains the average (like before), that idea was dropped?
