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(cost-estimator+proposer): split range based on optimism_safeHeadAtL1Block #256

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

ratankaliani
Copy link
Member

@ratankaliani ratankaliani commented Dec 4, 2024

To reduce derivation overhead, we can split the ranges based on the batches posted to L1 if the safeDb is enable on the rollup op-node RPC.

We can make use of optimism_safeHeadAtL1Block for doing this if it exists on the node.

Add these changes in both the cost-estimator and proposer.

Copy link

github-actions bot commented Dec 4, 2024

Metric Value
Batch Start 20,771,730
Batch End 20,771,735
Witness Generation (seconds) 96
Execution Duration (seconds) 78
Total Instruction Count 1,754,993,653
Oracle Verify Cycles 409,515,515
Derivation Cycles 817,828,279
Block Execution Cycles 457,565,578
Blob Verification Cycles 168,053,400
Total SP1 Gas 2,065,898,199
Number of Blocks 6
Number of Transactions 60
Ethereum Gas Used 26,389,504
Cycles per Block 292,498,942
Cycles per Transaction 29,249,894
Transactions per Block 10
Gas Used per Block 4,398,250
Gas Used per Transaction 439,825
BN Pair Cycles 0
BN Add Cycles 0
BN Mul Cycles 0
KZG Eval Cycles 0
EC Recover Cycles 1,409,682

@ratankaliani ratankaliani changed the title feat(cost-estimator): split range based on optimism_safeHeadAtL1Block feat(cost-estimator+proposer): split range based on optimism_safeHeadAtL1Block Dec 4, 2024
@ratankaliani ratankaliani merged commit ab0130a into main Dec 4, 2024
9 checks passed
@ratankaliani ratankaliani deleted the ratan/cost-estimator-split-ranges-smart branch December 4, 2024 22:10
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