Skip to content

feat: add benchmark-estimated-cost.yml workflow - #83

Merged
han0110 merged 1 commit into
mainfrom
han/feature/benchmark-estimated-cost
Sep 3, 2026
Merged

feat: add benchmark-estimated-cost.yml workflow#83
han0110 merged 1 commit into
mainfrom
han/feature/benchmark-estimated-cost

Conversation

@han0110

@han0110 han0110 commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

To report cost estimation diff on PRs if elf changes, benchmark on devnet latest 100 blocks

The report will look like:

Estimated guest cost

Measured over glamsterdam-devnet-8 blocks ending at 110,559.

Guest zkVM Base Head Change Peak heap (bytes)
reth openvm 432,567,329,621 434,722,228,032 +0.50% 12,962,955 -> 12,962,971
reth/openvm detail

Comparing 100 blocks.

Component Base Head Delta
precompile 130,689,301,432 130,689,319,882 +0.00%
rv64 201,278,078,525 202,666,134,062 +0.69%
system 100,599,949,664 101,366,774,088 +0.76%
total 432,567,329,621 434,722,228,032 +0.50%

Base ELF befa3b69743f targets v2.1.0-preview. Head ELF 9259d34e0148 targets v2.1.0-preview.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Estimated guest cost

No elf_sha256 changed in artifact-registry.json.

@han0110
han0110 force-pushed the han/feature/benchmark-estimated-cost branch 4 times, most recently from 4051bc8 to 2e580ff Compare September 1, 2026 12:00
@han0110
han0110 force-pushed the han/feature/benchmark-estimated-cost branch from 2e580ff to 21d632a Compare September 2, 2026 05:10
@han0110
han0110 requested a review from jsign September 2, 2026 13:36

@jsign jsign left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@han0110
han0110 merged commit dd6ac1a into main Sep 3, 2026
10 checks passed
@han0110
han0110 deleted the han/feature/benchmark-estimated-cost branch September 3, 2026 01:14
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