use partial gas price percentiles

Author: Tofel

seth/client.go

@@ -934,7 +934,7 @@ func (m *Client) CalculateGasEstimations(request GasEstimationRequest) GasEstima
 	defer cancel()
 
 	var disableEstimationsIfNeeded = func(err error) {
-		if strings.Contains(err.Error(), ZeroGasSuggestedErr) {
+		if strings.Contains(err.Error(), ZeroGasSuggestedErr) || strings.Contains(err.Error(), "Partial data received") {
 			L.Warn().Msg("Received incorrect gas estimations. Disabling them and reverting to hardcoded values. Remember to update your config!")
 			m.Cfg.Network.GasPriceEstimationEnabled = false
 		}

seth/gas.go

@@ -48,7 +48,8 @@ func (m *GasEstimator) Stats(fromNumber uint64, priorityPerc float64) (GasSugges
 	}
 	gasPercs, err := quantilesFromFloatArray(baseFees)
 	if err != nil {
-		return GasSuggestions{}, err
+		L.Debug().Err(err).Msg("Error calculating gas percentiles. Will use partial data.")
+		// return GasSuggestions{}, err
 	}
 	tips := make([]float64, 0)
 	for _, bf := range hist.Reward {
@@ -64,7 +65,8 @@ func (m *GasEstimator) Stats(fromNumber uint64, priorityPerc float64) (GasSugges
 	}
 	tipPercs, err := quantilesFromFloatArray(tips)
 	if err != nil {
-		return GasSuggestions{}, err
+		// return GasSuggestions{}, err
+		L.Debug().Err(err).Msg("Error calculating tip percentiles. Will use partial data.")
 	}
 	suggestedGasPrice, err := m.Client.Client.SuggestGasPrice(context.Background())
 	if err != nil {
@@ -105,23 +107,28 @@ type GasSuggestions struct {
 func quantilesFromFloatArray(fa []float64) (*GasPercentiles, error) {
 	perMax, err := stats.Max(fa)
 	if err != nil {
-		return nil, err
+		L.Debug().Err(err).Msg("Error calculating max gas price. Continuing")
+		// return nil, err
 	}
 	perc99, err := stats.Percentile(fa, 99)
 	if err != nil {
-		return nil, err
+		L.Debug().Err(err).Msg("Error calculating p99 gas price. Continuing")
+		// return nil, err
 	}
 	perc75, err := stats.Percentile(fa, 75)
 	if err != nil {
-		return nil, err
+		L.Debug().Err(err).Msg("Error calculating p75 gas price. Continuing")
+		// return nil, err
 	}
 	perc50, err := stats.Percentile(fa, 50)
 	if err != nil {
-		return nil, err
+		L.Debug().Err(err).Msg("Error calculating p50 gas price. Continuing")
+		// return nil, err
 	}
 	perc25, err := stats.Percentile(fa, 25)
 	if err != nil {
-		return nil, err
+		L.Debug().Err(err).Msg("Error calculating p25 gas price. Continuing")
+		// return nil, err
 	}
 	return &GasPercentiles{
 		Max:    perMax,

seth/gas_adjuster.go

@@ -467,22 +467,55 @@ func (m *Client) HistoricalFeeData(priority string) (baseFee float64, historical
 	stats, err := estimator.Stats(m.Cfg.Network.GasPriceEstimationBlocks, 99)
 	if err != nil {
 		L.Debug().
-			Msgf("Failed to get fee history due to: %s. Skipping automation gas estimation", err.Error())
+			Err(err).
+			Msgf("Failed to get fee history.. Skipping automation gas estimation")
 
 		return
 	}
 
 	switch priority {
 	case Priority_Degen:
+		if stats.GasPrice.Max == math.NaN() || stats.TipCap.Max == math.NaN() {
+			err = fmt.Errorf("Partial data received. Either base fee or suggested tip is 0 for Max values")
+			L.Debug().
+				Err(err).
+				Msgf("Failed to get fee history.. Skipping automation gas estimation")
+
+			return
+		}
 		baseFee = stats.GasPrice.Max
 		historicalGasTipCap = stats.TipCap.Max
 	case Priority_Fast:
+		if stats.GasPrice.Perc99 == math.NaN() || stats.TipCap.Perc99 == math.NaN() {
+			err = fmt.Errorf("Partial data received. Either base fee or suggested tip is 0 for Perc99 values")
+			L.Debug().
+				Err(err).
+				Msgf("Failed to get fee history.. Skipping automation gas estimation")
+
+			return
+		}
 		baseFee = stats.GasPrice.Perc99
 		historicalGasTipCap = stats.TipCap.Perc99
 	case Priority_Standard:
+		if stats.GasPrice.Perc50 == math.NaN() || stats.TipCap.Perc50 == math.NaN() {
+			err = fmt.Errorf("Partial data received. Either base fee or suggested tip is 0 for Perc50 values")
+			L.Debug().
+				Err(err).
+				Msgf("Failed to get fee history.. Skipping automation gas estimation")
+
+			return
+		}
 		baseFee = stats.GasPrice.Perc50
 		historicalGasTipCap = stats.TipCap.Perc50
 	case Priority_Slow:
+		if math.IsNaN(stats.GasPrice.Perc25) || math.IsNaN(stats.TipCap.Perc25) {
+			err = fmt.Errorf("Partial data received. Either base fee or suggested tip is 0 for Perc25 values")
+			L.Debug().
+				Err(err).
+				Msgf("Failed to get fee history.. Skipping automation gas estimation")
+
+			return
+		}
 		baseFee = stats.GasPrice.Perc25
 		historicalGasTipCap = stats.TipCap.Perc25
 	default: