edr icon indicating copy to clipboard operation
edr copied to clipboard

fix: elasticity multiplier for Base network

Open frangio opened this issue 8 months ago • 1 comments

Fixes https://github.com/NomicFoundation/edr/issues/887

Base fee parameters were a constant, this PR turns them into a function of chain id. This also prepares things for the ability to provide overrides for the parameters via config like with hardfork activations.

  • In calculate_next_base_fee_per_gas and PartialHeader::new they are now passed as an argument rather than taken from ChainSpecT.
  • Added Blockchain::base_fee_params.
  • In LocalBlockchain::new we get the constant params for the hardfork from ChainSpecT and store them in the struct.
  • In ForkedBlockchain::new we get the hardfork-variable params for the remote chain id and store them in the struct.

frangio avatar May 01 '25 23:05 frangio

⚠️ No Changeset found

Latest commit: 2d56caee82e8e1672abeb2ad8b049fbe6fc32fce

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar May 01 '25 23:05 changeset-bot[bot]

Closing as the fix will have to take a different shape: see https://github.com/NomicFoundation/edr/issues/887.

frangio avatar May 21 '25 15:05 frangio