Skip to content

Jhilliard/loadtest eip 1559 fixes#554

Merged
praetoriansentry merged 10 commits intomainfrom
jhilliard/loadtest-eip-1559-fixes
Mar 19, 2025
Merged

Jhilliard/loadtest eip 1559 fixes#554
praetoriansentry merged 10 commits intomainfrom
jhilliard/loadtest-eip-1559-fixes

Conversation

@praetoriansentry
Copy link
Copy Markdown
Member

Description

There are a few changes here that need to be tested and reviewed carefully:

  • Adding a new param, gas-price-multiplier which would allow the gas price to scaled
  • Fixing a panic when loadTestResults is empty
  • Refactoring to use configureTransactOpts everywhere and to read the suggested gas price from within that function only
  • Adding a new metric to the load test output that shows when the tester stopped sending requests
  • Switching from uniswapv3.Swapper to tokens.ERC20 in the pool / swap setting. The original token was not mintable which made reusing it very complicated
  • Adding some optimizations to speed up uniswapv3 mode when the contracts are already deployed
  • Adding some logs to help with using pre-deployed contracts

@praetoriansentry praetoriansentry marked this pull request as ready for review March 19, 2025 02:18
Copy link
Copy Markdown
Member

@leovct leovct left a comment

Choose a reason for hiding this comment

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

lgtm!!

Comment thread cmd/loadtest/uniswapv3/swapper.go Outdated
@praetoriansentry praetoriansentry merged commit d5a730f into main Mar 19, 2025
6 checks passed
@praetoriansentry praetoriansentry deleted the jhilliard/loadtest-eip-1559-fixes branch March 19, 2025 19:23
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