github-mirrors/VictoriaMetrics
1
0
Fork 0
mirror of https://github.com/VictoriaMetrics/VictoriaMetrics.git synced 2024-11-21 14:44:00 +00:00
Code Issues Projects Releases Packages Wiki Activity

app/vmselect/promql: fix corner-case calculation for ideriv

Browse source
This commit is contained in:
Aliaksandr Valialkin 2019-08-16 11:55:10 +03:00
parent 639b14e8ab
commit d6b5ed6d39
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/vmselect/promql/rollup.go
View file

@ -698,7 +698,7 @@ func rollupIderiv(rfa *rollupFuncArg) float64 {
// It is impossible to calculate derivative on 0 or 1 values.
return nan
}
return (values[0] - rfa.prevValue) / float64(timestamps[0]-rfa.prevTimestamp)
return (values[0] - rfa.prevValue) / (float64(timestamps[0]-rfa.prevTimestamp) * 1e-3)
}
vEnd := values[len(values)-1]
tEnd := timestamps[len(timestamps)-1]
@ -722,7 +722,7 @@ func rollupIderiv(rfa *rollupFuncArg) float64 {
}
dv := vEnd - vStart
dt := tEnd - tStart
return dv / (float64(dt) / 1000)
return dv / (float64(dt) * 1e-3)
}
func rollupChanges(rfa *rollupFuncArg) float64 {

2
app/vmselect/promql/rollup_test.go
View file

@ -56,7 +56,7 @@ func TestRollupIderivDuplicateTimestamps(t *testing.T) {
timestamps: []int64{100},
}
n = rollupIderiv(rfa)
if n != 0.5 {
if n != 500 {
t.Fatalf("unexpected value; got %v; want %v", n, 0.5)
}