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 (ff14a96) 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 (77adf7e) 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

This comment has been minimized.

@GunaDD GunaDD requested a review from shuklaayush December 20, 2025 01:19
@github-actions

This comment has been minimized.

@shuklaayush shuklaayush force-pushed the perf/new-keccak-run-reth branch from f7267b0 to f773644 Compare December 25, 2025 04:31
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@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 277 322,610 2,058,654 - - -
fibonacci 1,017 1,500,209 2,100,402 - - -
regex 2,352 4,137,502 17,695,216 - - -
ecrecover 741 122,859 2,262,772 - - -
pairing 1,454 1,745,742 25,408,302 - - -

Commit: ff14a96

Benchmark Workflow

@shuklaayush shuklaayush merged commit 019c81e into develop-new-keccak Dec 29, 2025
92 of 96 checks passed
@shuklaayush shuklaayush deleted the perf/new-keccak-run-reth branch December 29, 2025 19:39
shuklaayush added a commit that referenced this pull request Dec 29, 2025
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
shuklaayush added a commit that referenced this pull request Dec 29, 2025
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
jonathanpwang pushed a commit that referenced this pull request Dec 29, 2025
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
branch-rebase-bot bot pushed a commit that referenced this pull request Dec 30, 2025
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
branch-rebase-bot bot pushed a commit that referenced this pull request Jan 2, 2026
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
branch-rebase-bot bot pushed a commit that referenced this pull request Jan 5, 2026
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
branch-rebase-bot bot pushed a commit that referenced this pull request Jan 5, 2026
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
shuklaayush added a commit that referenced this pull request Jan 5, 2026
Closes INT-5779

---------

Co-authored-by: Ayush Shukla <ayush@axiom.xyz>
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