Skip to content

Conversation

2-towns
Copy link
Contributor

@2-towns 2-towns commented Apr 7, 2025

This PR adds a new command line argument to configure the maxPriorityFeePerGas available now with [EIP-1559 implementation] (codex-storage/nim-ethers#113)

@2-towns 2-towns force-pushed the feat/add-max-priority-fee-per-gas-command-arg branch from 041051c to be1ac9f Compare April 7, 2025 08:33
@2-towns 2-towns marked this pull request as ready for review April 10, 2025 08:49
@2-towns 2-towns force-pushed the feat/add-max-priority-fee-per-gas-command-arg branch from be1ac9f to c9e183c Compare May 28, 2025 14:17
@2-towns 2-towns force-pushed the feat/add-max-priority-fee-per-gas-command-arg branch 2 times, most recently from 86cd4db to 4ef9716 Compare June 3, 2025 14:22
@2-towns 2-towns requested review from markspanbroek and AuHau June 3, 2025 18:35
Copy link
Contributor

@markspanbroek markspanbroek left a comment

Choose a reason for hiding this comment

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

Nice, thanks @2-towns!

Copy link
Contributor

@AuHau AuHau left a comment

Choose a reason for hiding this comment

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

I was a bit hesitant to include some defaults that affect transaction "tips", but actually looking into EIP-1559 and checking the recent priority values on Gas tracker, I see that it is most probably needed in order to get transactions mined in a reasonable timeframe.

Just FYI. ethers.js is, I believe, using for the max priority fee 1,5 gwei as default value and not 1 gwei. To better understand it, I did some digging around this in Dune. For L2 the 1 gwei should be sufficient. The daily average for Arbitrum is bellow 0.1 gwei. You can have a look here: https://dune.com/queries/5234448/8604035/

Btw. on Ethreum mainnet it is between 1.5 to 2.6 gwei 🤪

So all in all, good work! Thanks for picking it up 👍

@AuHau
Copy link
Contributor

AuHau commented Jun 4, 2025

Btw. is EIP 1559 available on our PoA network? It is running old version of Geth, but I think it is the last version before The Merge, so it should be available as EIP 1559 was introduced before The Merge if I remember correctly?

But anyhow, nim-ethers will work correctly if EIP 1559 is not available right?

@2-towns
Copy link
Contributor Author

2-towns commented Jun 4, 2025

Btw. is EIP 1559 available on our PoA network? It is running old version of Geth, but I think it is the last version before The Merge, so it should be available as EIP 1559 was introduced before The Merge if I remember correctly?

Yes it should be available, I already checked with Slava, see genesis.json and how to run a private chain with EIP 1559 enabled.

But anyhow, nim-ethers will work correctly if EIP 1559 is not available right?

Yes exactly, it the EIP 1559 is not available, the "old" gas calculation will be used.

@2-towns 2-towns force-pushed the feat/add-max-priority-fee-per-gas-command-arg branch from 4ef9716 to 06600e7 Compare June 5, 2025 06:16
@2-towns 2-towns enabled auto-merge June 5, 2025 06:18
@2-towns 2-towns added this pull request to the merge queue Jun 5, 2025
Merged via the queue into master with commit 3e17207 Jun 5, 2025
18 checks passed
@2-towns 2-towns deleted the feat/add-max-priority-fee-per-gas-command-arg branch June 5, 2025 09:03
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.

4 participants