Skip to content

feat: RecordSeeker::get_aligned_record_size#1863

Merged
arayikhalatyan merged 1 commit into
feat/new-executionfrom
feat/recordseeker-recordsize
Jul 14, 2025
Merged

feat: RecordSeeker::get_aligned_record_size#1863
arayikhalatyan merged 1 commit into
feat/new-executionfrom
feat/recordseeker-recordsize

Conversation

@arayikhalatyan

@arayikhalatyan arayikhalatyan commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@stephenh-axiom-xyz stephenh-axiom-xyz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@arayikhalatyan arayikhalatyan merged commit 9315f1a into feat/new-execution Jul 14, 2025
32 of 33 checks passed
@arayikhalatyan arayikhalatyan deleted the feat/recordseeker-recordsize branch July 14, 2025 20:13
@github-actions

Copy link
Copy Markdown
group app.proof_time_ms app.cycles app.cells_used leaf.proof_time_ms leaf.cycles leaf.cells_used
verify_fibair (+31 [+2.5%]) 1,286 322,699 17,271,190 - - -
fibonacci (-16 [-0.7%]) 2,357 1,500,277 50,589,231 - - -
regex (+50 [+0.7%]) 7,034 4,165,432 166,449,586 - - -
ecrecover (+27 [+2.0%]) 1,377 137,283 8,179,549 - - -
pairing (+39 [+1.0%]) 4,022 1,862,964 102,530,853 - - -

Commit: bb04fc6

Benchmark Workflow

@codspeed-hq

codspeed-hq Bot commented Jul 14, 2025

Copy link
Copy Markdown

CodSpeed WallTime Performance Report

Merging #1863 will not alter performance

Comparing feat/recordseeker-recordsize (bb04fc6) with feat/new-execution (48c80d7)

⚠️ 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

✅ 20 untouched benchmarks

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