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.

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

Estimate gas for withdrawals #27

Closed
wants to merge 10 commits into from
Closed

Estimate gas for withdrawals #27

wants to merge 10 commits into from

Conversation

dapplion
Copy link
Member

Results with current commit and setOnWithdrawalsUnwrapToGNOByDefault == true

item count gasUsed gas / w
success withdrawal 1 106440 106440
success withdrawal 2 155905 77953
success withdrawal 4 269835 67459
success withdrawal 8 497707 62213
success withdrawal 16 953429 59589
success withdrawal 32 1864876 58277
success withdrawal 64 3687782 57622
failed withdrawal 1 112103 112103
failed withdrawal 2 160640 80320
failed withdrawal 4 287714 71929
failed withdrawal 8 541875 67734
failed withdrawal 16 1050177 65636
failed withdrawal 32 2066790 64587
failed withdrawal 64 4100056 64063
retry fail withdrawal 1 50353 50353
retry fail withdrawal 2 50353 25177
retry fail withdrawal 4 50353 12588
retry fail withdrawal 8 50353 6294
retry fail withdrawal 16 50353 3147
retry fail withdrawal 32 50353 1574
retry fail withdrawal 64 50353 787
retry success withdrawal 1 106440 106440
retry success withdrawal 2 155905 77953
retry success withdrawal 4 269835 67459
retry success withdrawal 8 497707 62213
retry success withdrawal 16 953429 59589
retry success withdrawal 32 1864876 58277
retry success withdrawal 64 3687782 57622

@dapplion dapplion mentioned this pull request Feb 25, 2023
Closed
Barichek
Barichek reviewed Mar 10, 2023
View reviewed changes
Copy link
Contributor

@Barichek Barichek left a comment

Choose a reason for hiding this comment

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

Cool!

@dapplion dapplion changed the base branch from ib-withdrawal-contract-part to master March 30, 2023 01:48
@dapplion dapplion mentioned this pull request May 10, 2023
Merged
@dapplion
Copy link
Member Author

Contract design is final, no need to keep this gas estimations up

@dapplion dapplion closed this Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Barichek Barichek Barichek left review comments

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
@dapplion @Barichek