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

Use saved gas parameters for new gas algorithm #187

Closed
Tracked by #279
arhag opened this issue Jan 23, 2024 · 0 comments · Fixed by #197
Closed
Tracked by #279

Use saved gas parameters for new gas algorithm #187

arhag opened this issue Jan 23, 2024 · 0 comments · Fixed by #197
Assignees

Comments

@arhag
Copy link
Member

arhag commented Jan 23, 2024

Depends on #186 and eosnetworkfoundation/silkworm#122.

Use the gas parameters stored associated with each block (see #186) to pass into the silkworm infrastructure developed in eosnetworkfoundation/silkworm#122 so that evmc can use the appropriate gas parameters when tracking consumed gas.

Update integration test to show how gas parameters can be updated.

Add an integration test to show how gas parameters are handled under forks.

@github-project-automation github-project-automation bot moved this to Todo in EOS EVM Jan 23, 2024
@yarkinwho yarkinwho changed the title Process config change event to pass gas parameters into ExecutionState Process config change event to save gas parameters Jan 23, 2024
@arhag arhag changed the title Process config change event to save gas parameters Use saved gas parameters for new gas algorithm Jan 24, 2024
@elmato elmato moved this from Todo to In Progress in EOS EVM Feb 26, 2024
@elmato elmato linked a pull request Mar 20, 2024 that will close this issue
@elmato elmato moved this from In Progress to Awaiting Review in EOS EVM Mar 20, 2024
@github-project-automation github-project-automation bot moved this from Awaiting Review to Done in EOS EVM Mar 20, 2024
@stephenpdeos stephenpdeos added this to the EVM Node 1.0.0-rc1 milestone May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants