Skip to content

hide: LAN-52 add benchmark display executable#17

Merged
winitzki merged 2 commits intomainfrom
sergeiwinitzki/lan-51-add-benchmark-display
Mar 26, 2026
Merged

hide: LAN-52 add benchmark display executable#17
winitzki merged 2 commits intomainfrom
sergeiwinitzki/lan-51-add-benchmark-display

Conversation

@winitzki
Copy link
Copy Markdown
Contributor

@winitzki winitzki commented Mar 24, 2026

This PR adds a CLI tool that renders benchmark graphs to a standalone HTML file (using the same Plotly graph style as the web UI) and opens it locally.

The data is taken from JSON files that the benchmark outputs.
In this way, local development can be faster - each benchmark graph can be visualized on the local machine in the same way it will appear in CI, without waiting for CI.

This will work if the benchmark writes its data as JSON. Currently the full benchmark run does this (bench.sh) as well as individual benchmarks: bench-csv-table.sh and bench-complex-spec.sh.

The tool can take several JSON files (several benchmark runs) and plot the results as lines. If JSON files have git hashes in their names then the git hashes will be used for the graph legends.

Alternatively, data can be taken from the local sqlite3 database (downloaded from brossa.io).

@winitzki winitzki changed the title hide: add benchmark display executable hide: LAN-52 add benchmark display executable Mar 24, 2026
Copy link
Copy Markdown
Contributor

@harryroscoe-artificial harryroscoe-artificial left a comment

Choose a reason for hiding this comment

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

Nice

Copy link
Copy Markdown
Contributor

@chrisdone-artificial chrisdone-artificial left a comment

Choose a reason for hiding this comment

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

Nice!

@winitzki winitzki merged commit e87d02f into main Mar 26, 2026
1 check passed
@winitzki winitzki deleted the sergeiwinitzki/lan-51-add-benchmark-display branch March 26, 2026 08:37
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.

3 participants