Skip to content

docs: SDK standard test with p256 + doc update #2066

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
Merged
merged 10 commits into from
Aug 28, 2025
Merged

docs: SDK standard test with p256 + doc update #2066

merged 10 commits into from
Aug 28, 2025

Conversation

stephenh-axiom-xyz
Copy link
Contributor

@stephenh-axiom-xyz stephenh-axiom-xyz commented Aug 26, 2025 •
edited
Loading

Resolves INT-4574.

@stephenh-axiom-xyz stephenh-axiom-xyz marked this pull request as ready for review August 26, 2025 20:23
jonathanpwang
jonathanpwang reviewed Aug 26, 2025
View reviewed changes
jonathanpwang

This comment was marked as resolved.

Sign in to view

@stephenh-axiom-xyz
chore: SDK standard test that uses p256
0cd5d4e 
chore: resolve some memory usage items + lint errors

chore: CI fixes

chore: please work...

chore: another try

chore: yikes

chore: ...
@stephenh-axiom-xyz
docs: SDK standard config example
8506b6a
@stephenh-axiom-xyz
chore: move p256 example to sdk
9a20507
@stephenh-axiom-xyz stephenh-axiom-xyz force-pushed the chore/sdk-standard-examples branch from 7012353 to 9a20507 Compare August 27, 2025 13:44
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@stephenh-axiom-xyz stephenh-axiom-xyz changed the title chore: SDK standard test that uses p256 + example docs: SDK standard test with p256 + doc update Aug 27, 2025
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
jonathanpwang
jonathanpwang reviewed Aug 27, 2025
View reviewed changes
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
jonathanpwang
jonathanpwang reviewed Aug 28, 2025
View reviewed changes
jonathanpwang
jonathanpwang approved these changes Aug 28, 2025
View reviewed changes
Copy link
Contributor

@jonathanpwang jonathanpwang left a comment

Choose a reason for hiding this comment

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

LGTM after addressing last comment

@github-actions 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 (+34 [+12.8%]) 299 322,610 2,058,654 - - -
fibonacci (-6 [-0.6%]) 1,015 1,500,210 1,060,232 - - -
regex (+122 [+4.2%]) 3,047 4,108,483 13,403,604 - - -
ecrecover (+43 [+3.7%]) 1,191 140,497 2,275,056 - - -
pairing (+5 [+0.3%]) 1,615 1,882,939 23,722,622 - - -

Commit: 8d7e409

Benchmark Workflow

@stephenh-axiom-xyz stephenh-axiom-xyz merged commit 10acf91 into main Aug 28, 2025
26 of 27 checks passed
@stephenh-axiom-xyz stephenh-axiom-xyz deleted the chore/sdk-standard-examples branch August 28, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonathanpwang jonathanpwang jonathanpwang approved these changes

@yi-sun yi-sun Awaiting requested review from yi-sun

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
@stephenh-axiom-xyz @jonathanpwang