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.

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

zkvm proof size statistics data #852

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

hero78119
Copy link
Collaborator

@hero78119 hero78119 commented Mar 5, 2025

This PR implement formatter on ZKVMProof to break down proof size, which helps to identify and prioritize recursion tasks

NOTE: it's based on basefold mpcs

e2e proof stat: overall_size 19.07mb.
opcode mpcs commitment 0%
opcode mpcs opening 40%
opcode tower proof 0%
opcode main sumcheck proof 0%
table mpcs commitment 0%
table mpcs opening 32%
table mpcs fixed opening 25%
table tower proof 0%
table same r sumcheck proof 0%

@hero78119 hero78119 requested a review from lispc March 5, 2025 09:17
@hero78119 hero78119 force-pushed the feat/statistics_proof_size branch from 89ddd9f to c3ce926 Compare March 5, 2025 09:22
@hero78119 hero78119 changed the title statistics data for proof size zkvm proof size statistics data Mar 5, 2025
@hero78119 hero78119 force-pushed the feat/statistics_proof_size branch from c3ce926 to e3b2fc6 Compare March 5, 2025 09:25
@hero78119 hero78119 enabled auto-merge March 5, 2025 10:06
@hero78119 hero78119 added this pull request to the merge queue Mar 5, 2025
Merged via the queue into scroll-tech:master with commit e8197db Mar 5, 2025
4 checks passed
@hero78119 hero78119 deleted the feat/statistics_proof_size branch March 5, 2025 13:12
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