Skip to content

Conversation

@GunaDD
Copy link
Contributor

@GunaDD GunaDD commented Dec 19, 2025

Closes INT-5779

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 19, 2025

CodSpeed Performance Report

Merging #2328 will not alter performance

Comparing perf/new-keccak-run-reth (02c19f5) with main (a1bce3e)1

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 24 untouched
⏩ 36 skipped2

Footnotes

  1. No successful run was found on develop-new-keccak (ad8af10) during the generation of this report, so main (a1bce3e) was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 36 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@GunaDD GunaDD force-pushed the perf/new-keccak-run-reth branch from 687a6ca to ccb6238 Compare December 19, 2025 22:41
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@GunaDD GunaDD force-pushed the perf/new-keccak-run-reth branch from 2bd040d to 592b485 Compare December 19, 2025 23:31
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@GunaDD GunaDD force-pushed the perf/new-keccak-run-reth branch from 5d9cbdd to 36849f5 Compare December 20, 2025 00:49
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@GunaDD
Copy link
Contributor Author

GunaDD commented Dec 20, 2025

Reth run panicked due to a

failed to execute client: MptError(NodeRefMismatch)
failed to execute client: MptError(NodeRefMismatch)

error and it is unclear if it is due to the new keccak or not but @shuklaayush mentioned he will take a look. @shuklaayush can merge the PR if he likes to.

https://github.com/axiom-crypto/openvm-reth-benchmark/actions/runs/20386012415/job/58586939468

@github-actions
Copy link

group app.proof_time_ms app.cycles app.cells_used leaf.proof_time_ms leaf.cycles leaf.cells_used
verify_fibair 223 322,658 2,058,654 - - -
fibonacci 1,014 1,500,209 2,100,402 - - -
regex 2,359 4,137,502 17,695,216 - - -
ecrecover 742 122,859 2,263,998 - - -
pairing 1,474 1,745,742 25,408,302 - - -

Commit: 02c19f5

Benchmark Workflow

@GunaDD GunaDD requested a review from shuklaayush December 20, 2025 01:19
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.

2 participants