Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

eth/gasprice: improve stability of estimated price #22722

Merged
merged 13 commits into from
Apr 28, 2021
Merged

eth/gasprice: improve stability of estimated price #22722

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
d133bad
remove outliers in gpo calc
GregTheGreek Apr 22, 2021
9635379
add sort and test
GregTheGreek Apr 22, 2021
4716b14
udpdate comment
GregTheGreek Apr 22, 2021
adbaeaa
wip
GregTheGreek Apr 22, 2021
0d7e77b
cleanup formula
GregTheGreek Apr 22, 2021
dbe5de1
cleanup comments and variables
GregTheGreek Apr 22, 2021
014804b
update per comments
GregTheGreek Apr 22, 2021
647b203
fix formulas, add tests
GregTheGreek Apr 22, 2021
40939f3
Merge pull request #1 from GregTheGreek/greg/gpo-fix-1
GregTheGreek Apr 22, 2021
e617d50
fix test
GregTheGreek Apr 23, 2021
be9812f
remove 0 gas transactions
GregTheGreek Apr 23, 2021
9ad1997
simplify change
GregTheGreek Apr 26, 2021
e797c53
revert back to 3 sampled txs
GregTheGreek Apr 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions eth/gasprice/gasprice.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,6 +199,9 @@ func (gpo *Oracle) getBlockPrices(ctx context.Context, signer types.Signer, bloc

var prices []*big.Int
for _, tx := range txs {
if tx.GasPriceIntCmp(common.Big1) <= 0 {
continue
}
sender, err := types.Sender(signer, tx)
if err == nil && sender != block.Coinbase() {
prices = append(prices, tx.GasPrice())
Expand Down