forked from ethereum/go-ethereum
-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Sequencer Fee Pricing Part 1, fix: revive the binary-search-enabled D…
…oEstimateGas from geth (ethereum#276) * fix: add nil check when estimating gas to avoid panic during contract creation * fix: revert DoEstimateGas changes introduced as a temp fix in ethereum#22 * Tweak fudge factor for Geth gas estimation (ethereum#292) * experimenting with gas fudge factor * fix formatting * try using 1m gas constant * Add log statement for the gas estimate * Add more detail to gas estimate log * one more log edit * Try new formula * Bump base gas * Even more base gas * even more * Just 1m? * one more time * Final cleanup * Minor fix * Update internal/ethapi/api.go * don't use cap-1 * Make sure data is not nil Co-authored-by: Karl Floersch <karl@karlfloersch.com> Co-authored-by: Ben Jones <ben@pseudonym.party> Co-authored-by: smartcontracts <kelvinfichter@gmail.com> Co-authored-by: Karl Floersch <karl@karlfloersch.com>
- Loading branch information
1 parent
be44f8d
commit f7b384f
Showing
2 changed files
with
84 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters