Skip to content

Sort results by average change in gas_diff_stats.py#14888

Merged
cameel merged 1 commit intodevelopfrom
gas-diff-stats-sort-by-avg
Feb 27, 2024
Merged

Sort results by average change in gas_diff_stats.py#14888
cameel merged 1 commit intodevelopfrom
gas-diff-stats-sort-by-avg

Conversation

@cameel
Copy link
Collaborator

@cameel cameel commented Feb 26, 2024

A simple tweak get tables sorted like in #14887 (comment)

I.e. the rows are now ordered by average percentage change, which makes it much easier to find the relevant rows and see the general trend, especially when the table is long.

@cameel cameel requested a review from r0qs February 26, 2024 19:27
@cameel cameel self-assigned this Feb 26, 2024
Copy link
Member

@r0qs r0qs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! LGTM, just one comment.

@cameel cameel force-pushed the gas-diff-stats-sort-by-avg branch from 83fd599 to 180f744 Compare February 26, 2024 20:58
@cameel cameel enabled auto-merge February 26, 2024 21:02
@cameel cameel disabled auto-merge February 26, 2024 21:02
@cameel cameel merged commit 14e217e into develop Feb 27, 2024
@cameel cameel deleted the gas-diff-stats-sort-by-avg branch February 27, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants