Skip to content

Formatting of stats output #144

@spier

Description

@spier

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?

Screenshot

Screenshot 2023-10-16 at 22 08 17

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions