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

partial fill block + UX improvements #59

Merged
merged 15 commits into from
Dec 12, 2024
Merged

partial fill block + UX improvements #59

merged 15 commits into from
Dec 12, 2024

Conversation

zeroXbrock
Copy link
Member

@zeroXbrock zeroXbrock commented Dec 10, 2024

  • remove timeout from spammer
  • add env var C_FILL_PERCENT to fill blocks up to a given percent when the fill-blocks scenario is run with contender run
  • add stress.toml, tweak mempool.toml
  • cleanup logs
  • don't crash on failed tasks
  • graceful CTRL-C termination (so you can get your run_id even if you need to terminate)
  • accurately estimate gas limits (differ by calldata)

Motivation

partial fill-block:

in case you want to fill a block up to <100%

the rest:

  • don't crash the whole run because of one failed tx
  • stop early and still save results
  • get more info about your txs and their execution status
  • accurate gas estimates mean more txs will fit in a block
  • new scenario spams more opcodes (even more are available in the contract)

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@zeroXbrock zeroXbrock changed the title partial fill block partial fill block + UX improvements Dec 11, 2024
@zeroXbrock zeroXbrock merged commit ac8f299 into main Dec 12, 2024
5 checks passed
@zeroXbrock zeroXbrock deleted the partial-fill-block branch December 12, 2024 00:20
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