Skip to content

add benchmark printout ordering feature - #1446

Open
baltzell wants to merge 5 commits into
developmentfrom
benchmark-csv-2
Open

baltzell wants to merge 5 commits into
developmentfrom
benchmark-csv-2

Conversation

@baltzell

@baltzell baltzell commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

This new sortByName() overwrites its Benchmark's LinkedHashMap<String,BenchmarkTimer> storage container, to achive two things for printing:

  1. deterministic order, based on String naturals
  2. put some "special" keys at the beginning

For this to have any effect, one needs to call sortByName before printing. Could instead have different benchmarks with different automatic sorting. Currently the default ordering is the call order, which is a good default and what you want for serial stuff.

@baltzell baltzell changed the title add option to reorder benchmarks add option to prioritize certain benchmarks in printout Sep 21, 2026
@baltzell
baltzell enabled auto-merge (squash) September 21, 2026 20:05
@baltzell
baltzell requested a review from c-dilks September 21, 2026 23:31
@baltzell baltzell changed the title add option to prioritize certain benchmarks in printout add benchmark printout ordering feature Sep 21, 2026
@baltzell
baltzell marked this pull request as draft September 22, 2026 00:01
auto-merge was automatically disabled September 22, 2026 00:01

Pull request was converted to draft

@baltzell
baltzell marked this pull request as ready for review September 22, 2026 15:02
@baltzell
baltzell enabled auto-merge (squash) September 22, 2026 15:02

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant