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

Tracking issue: Reduce the number of cycles for the zk prover. #1271

Open
1 of 4 tasks
theochap opened this issue Jan 2, 2024 · 2 comments
Open
1 of 4 tasks

Tracking issue: Reduce the number of cycles for the zk prover. #1271

theochap opened this issue Jan 2, 2024 · 2 comments
Assignees

Comments

@theochap
Copy link
Contributor

theochap commented Jan 2, 2024

Description
This is a tracking issue to bring down the number of cycles from the zk-prover's main process (ie all the operations outside of signature computation).

Potential areas of improvement

Current stats for the demo-prover:

  • Standard hash function patched with risc0/rust_crypto
  • Signature verification patched

Block stats

Description Value
Total blocks 3
Blocks with transactions 3
Number of blobs 3
Total number of transactions 14
Average number of transactions per block 1

Cycle Metrics

Function Average Cycles Num Calls
Cycles per block 5122643 3
apply_blob 3932560 3
verify 749219 4
end_slot 648684 3
compute_state_update 610264 3
begin_slot 91216 3
@theochap theochap self-assigned this Jan 2, 2024
@citizen-stig
Copy link
Member

I thought that signature verification was patched in scope of #1245

@theochap
Copy link
Contributor Author

theochap commented Jan 2, 2024

I thought that signature verification was patched in scope of #1245

@preston-evans98 ?

@theochap theochap changed the title Tracking issue: Reducing the number of cycles for the zk prover. Tracking issue: Reduce the number of cycles for the zk prover. Jan 2, 2024
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

No branches or pull requests

2 participants