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

feat(proposer): remove manual network proof timeout #250

Merged
merged 1 commit into from
Dec 1, 2024

Conversation

ratankaliani
Copy link
Member

The proof timeout logic was necessary when we could not assume that the network would correctly set the status of the proof to failed, and the application (op-succinct) had to do this itself to re-request a proof.

This inadvertently causes a backlogged cluster to become even more backlogged.

Ex.
Requested proofs on the cluster (which are capped at maxConcurrentProofRequests per proposer) have been generating for > proofTimeout seconds (e.g. 4 hours). The application "times out" the proofs in it's own state, and re-requests the proofs, putting more load on the network.

@ratankaliani ratankaliani changed the title feat(proposer): remove network proof timeout feat(proposer): remove manual network proof timeout Dec 1, 2024
@ratankaliani ratankaliani merged commit ef62b06 into main Dec 1, 2024
8 checks passed
@ratankaliani ratankaliani deleted the ratan/remove-network-proof-timeout branch December 1, 2024 17:22
Copy link

github-actions bot commented Dec 1, 2024

Metric Value
Batch Start 20,634,930
Batch End 20,634,935
Witness Generation (seconds) 53
Execution Duration (seconds) 40
Total Instruction Count 1,332,721,970
Oracle Verify Cycles 289,812,086
Derivation Cycles 838,330,805
Block Execution Cycles 149,221,456
Blob Verification Cycles 101,677,936
Total SP1 Gas 1,585,430,164
Number of Blocks 6
Number of Transactions 37
Ethereum Gas Used 12,714,916
Cycles per Block 222,120,328
Cycles per Transaction 36,019,512
Transactions per Block 6
Gas Used per Block 2,119,152
Gas Used per Transaction 343,646
BN Pair Cycles 0
BN Add Cycles 0
BN Mul Cycles 0
KZG Eval Cycles 0
EC Recover Cycles 322,016

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.

1 participant