fix SummarizeValues to return last non-NaN value

go-graphite · Nov 29, 2024 · 8a550a5 · 8a550a5
1 parent cdab347
commit 8a550a5
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/expr/consolidations/consolidations.go b/expr/consolidations/consolidations.go
@@ -188,7 +188,13 @@ func SummarizeValues(f string, values []float64, XFilesFactor float32) float64 {
 			}
 		}
 	case "last", "current":
-		rv = values[len(values)-1]
+		rv = math.NaN()
+		for i := len(values) - 1; i >= 0; i-- {
+			if !math.IsNaN(values[i]) {
+				rv = values[i]
+				break
+			}
+		}
 		total = notNans(values)
 	case "range", "rangeOf":
 		vMax := math.Inf(-1)