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: params estimator for delegate action #8596

Closed
wants to merge 2 commits into from

Conversation

jakmeier
Copy link
Contributor

The main cost of a meta transaction is for all the inner action costs.
The send costs for them are paid twice, which covers all the extra checks.

However, there is also a bit of constant overhead for each delegate action.
One very small bit when sending it, a slightly larger bit when executing it.

This adds the estimation code for this to the parameter estimator.

Note: All estimations were run on top of the changes from #8578
because that actually changes the signature verification, which happens
to be 99% of the cost we measure here.

@jakmeier jakmeier mentioned this pull request Feb 17, 2023
4 tasks
@jakmeier
Copy link
Contributor Author

Keeping it as a draft for now, since the results are still quite unstable / high-variance. I'd like to improve on that, but OTOH we already get useful numbers from this implementation and can basically move ahead with meta transaction stabilization.

For estimation results, see this comment in the meta tx gas cost estimation issue: #8114 (comment)

@akhi3030
Copy link
Collaborator

akhi3030 commented Nov 3, 2023

I think this is not being actively worked on.

@akhi3030 akhi3030 closed this Nov 3, 2023
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