Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

1.5x the estimated gas when sending a message #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tkporter
Copy link
Contributor

Seems like Goerli gas estimation has been a bit weird.

Estimating gas against the ankr public url (this is what the SDK uses):

$ cast estimate --rpc-url https://rpc.ankr.com/eth_goerli 0xDDcFEcF17586D08A5740B7D91735fcCE3dfe3eeD 'dispatch(uint32,bytes32,bytes)' 80001 0x000000000000000000000000bc3cfeca7df5a45d61bc60e7898e63670e1654ae 0x4d595f4d657373616765
34470

With a different public url:

$ cast estimate --rpc-url https://eth-goerli.public.blastapi.io 0xDDcFEcF17586D08A5740B7D91735fcCE3dfe3eeD 'dispatch(uint32,bytes32,bytes)' 80001 0x000000000000000000000000bc3cfeca7df5a45d61bc60e7898e63670e1654ae 0x4d595f4d657373616765
47666

Against anvil -f https://rpc.ankr.com/eth_goerli:

$ cast estimate --rpc-url http://localhost:8545 0xDDcFEcF17586D08A5740B7D91735fcCE3dfe3eeD 'dispatch(uint32,bytes32,bytes)' 80001 0x000000000000000000000000bc3cfeca7df5a45d61bc60e7898e63670e1654ae 0x4d595f4d657373616765
49428

Actual gas used against the anvil fork: 47432

So for now, just going to overestimate gas

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant