Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(utils/client): add secp256r1 cycle tracking #324

Merged
merged 1 commit into from
Jan 14, 2025
Merged

feat(utils/client): add secp256r1 cycle tracking #324

merged 1 commit into from
Jan 14, 2025

Conversation

fakedev9999
Copy link
Member

No description provided.

@fakedev9999 fakedev9999 force-pushed the taehoon/add-secp256r1-tracking branch from 8b66383 to 361ac33 Compare January 14, 2025 03:52
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 14, 2025 •
edited
Loading

Metric Value
Batch Start 22,514,130
Batch End 22,514,135
Witness Generation (seconds) 81
Execution Duration (seconds) 71
Total Instruction Count 2,468,252,329
Oracle Verify Cycles 537,429,163
Derivation Cycles 956,892,535
Block Execution Cycles 808,782,679
Blob Verification Cycles 236,850,490
Total SP1 Gas 0
Number of Blocks 6
Number of Transactions 165
Ethereum Gas Used 38,605,138
Cycles per Block 411,375,388
Cycles per Transaction 14,959,105
Transactions per Block 27
Gas Used per Block 6,434,189
Gas Used per Transaction 233,970
BN Pair Cycles 0
BN Add Cycles 0
BN Mul Cycles 0
KZG Eval Cycles 0
EC Recover Cycles 737,894
P256 Verify Cycles 0

@fakedev9999 fakedev9999 force-pushed the taehoon/add-secp256r1-tracking branch from 361ac33 to 5284ea2 Compare January 14, 2025 05:19
ratankaliani
ratankaliani approved these changes Jan 14, 2025
View reviewed changes
Copy link
Member

@ratankaliani ratankaliani left a comment

Choose a reason for hiding this comment

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

LGTM

@fakedev9999 fakedev9999 marked this pull request as ready for review January 14, 2025 06:36
@fakedev9999 fakedev9999 merged commit 1095e9c into main Jan 14, 2025
8 checks passed
@fakedev9999 fakedev9999 deleted the taehoon/add-secp256r1-tracking branch January 14, 2025 07:10
leruaa pushed a commit that referenced this pull request Jan 15, 2025
@fakedev9999 @leruaa
feat(utils/client): add secp256r1 cycle tracking (#324)
903d0b0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ratankaliani ratankaliani ratankaliani approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fakedev9999 @ratankaliani