Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Golovanov399
Copy link
Contributor

@Golovanov399 Golovanov399 commented Aug 19, 2025

towards INT-4744

@codspeed-hq
Copy link

codspeed-hq bot commented Aug 19, 2025

CodSpeed WallTime Performance Report

Merging #2008 will degrade performances by 11.53%

Comparing chore/migrate-native-ext-cuda (11a28f2) with main (38f1057)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

⚡ 2 improvements
❌ 2 regressions
✅ 20 untouched benchmarks
🆕 3 new benchmarks
⁉️ 6 dropped benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
benchmark_execute[factorial_iterative_u256] 149.7 ms 166.8 ms -10.25%
benchmark_execute[fibonacci_recursive] 76 ms 66.2 ms +14.84%
benchmark_execute[sha256_iter] 110 ms 99.7 ms +10.29%
benchmark_execute_metered[revm_snailtracer] 15.5 ms 17.5 ms -11.53%
⁉️ benchmark_internal_verifier_execute[fibonacci] 46.2 ms N/A N/A
⁉️ benchmark_internal_verifier_execute_metered[fibonacci] 69 ms N/A N/A
⁉️ benchmark_internal_verifier_execute_preflight[fibonacci] 171.8 ms N/A N/A
🆕 benchmark_leaf_verifier_execute N/A 752.7 ms N/A
⁉️ benchmark_leaf_verifier_execute[kitchen-sink] 747.3 ms N/A N/A
🆕 benchmark_leaf_verifier_execute_metered N/A 961.5 ms N/A
⁉️ benchmark_leaf_verifier_execute_metered[kitchen-sink] 966.5 ms N/A N/A
🆕 benchmark_leaf_verifier_execute_preflight N/A 1.9 s N/A
⁉️ benchmark_leaf_verifier_execute_preflight[kitchen-sink] 1.9 s N/A N/A

Footnotes

  1. No successful run was found on feat/tracegen-gpu (07ab284) during the generation of this report, so main (38f1057) was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@github-actions

This comment has been minimized.

@Golovanov399 Golovanov399 force-pushed the chore/migrate-native-ext-cuda branch from 2ddfccc to f1f2d16 Compare August 20, 2025 13:50
@Golovanov399 Golovanov399 changed the base branch from feat/tracegen-gpu to feat/gpu-test-builder August 20, 2025 13:52
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Base automatically changed from feat/gpu-test-builder to feat/tracegen-gpu August 20, 2025 16:38
@Golovanov399 Golovanov399 force-pushed the chore/migrate-native-ext-cuda branch from d4a9efe to cddf998 Compare August 20, 2025 16:48
@github-actions

This comment has been minimized.

@Golovanov399 Golovanov399 changed the title [wip] chore: migrate native extensions chore: migrate native extensions Aug 20, 2025
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@Golovanov399 Golovanov399 force-pushed the chore/migrate-native-ext-cuda branch from 2da54d7 to 39d6c6a Compare August 21, 2025 13:33
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@Golovanov399 Golovanov399 marked this pull request as ready for review August 21, 2025 14:15
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Alexander Golovanov added 2 commits August 21, 2025 16:42
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Alexander Golovanov added 2 commits August 21, 2025 17:23
@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.

Alexander Golovanov added 2 commits August 21, 2025 18:10
@github-actions

This comment has been minimized.

Copy link
Contributor

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

Choose a reason for hiding this comment

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

LGTM

@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 2,112 322,700 18,750,324 - - -
fibonacci 2,429 1,500,210 51,504,507 - - -
regex 7,545 4,108,597 164,734,992 - - -
ecrecover 1,407 140,487 8,866,654 - - -
pairing 3,906 1,882,939 98,834,293 - - -

Commit: 11a28f2

Benchmark Workflow

@Golovanov399 Golovanov399 merged commit 4183e3e into feat/tracegen-gpu Aug 21, 2025
36 of 40 checks passed
@Golovanov399 Golovanov399 deleted the chore/migrate-native-ext-cuda branch August 21, 2025 19:24
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.

4 participants