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

Optimize execution gas usage #53

Open
mpetrun5 opened this issue Apr 4, 2024 · 1 comment
Open

Optimize execution gas usage #53

mpetrun5 opened this issue Apr 4, 2024 · 1 comment

Comments

@mpetrun5
Copy link

mpetrun5 commented Apr 4, 2024

Currently, we fetch and prove storage roots each proposal which costs per proposal when batched about 400k gas.
We need to avoid that cost and calculate that only once.

Implementation details

Calculate the necessary storage root before looping through the proposals and use that to verify the proposal with.
All the proposals should have the same source and security model.

Testing details

  • unit tests

Acceptance Criteria

  • cost when batching the next proposal should only include the cost to verify the proof against the precalculated storage root
@mpetrun5
Copy link
Author

mpetrun5 commented Apr 4, 2024

This should be tackled after the Axiom refactor that would change the executor logic to use storage roots directly instead of calculating them from the state root.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant