Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Unified Diff: perf/go/parser/funcs.go

Issue 610793003: Add ratio calculation to take the ratio of two calculations. (Closed) Base URL: https://skia.googlesource.com/buildbot@master
Patch Set: remove debug code Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | perf/go/parser/parser.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: perf/go/parser/funcs.go
diff --git a/perf/go/parser/funcs.go b/perf/go/parser/funcs.go
index bca448e00a1770f894ff72a174e725aca02fff43..26870a4b460953e5e58a59f57965ad18f0522a5f 100644
--- a/perf/go/parser/funcs.go
+++ b/perf/go/parser/funcs.go
@@ -2,6 +2,7 @@ package parser
import (
"fmt"
+ "math"
"net/url"
"strconv"
@@ -175,3 +176,37 @@ func (AveFunc) Describe() string {
}
var aveFunc = AveFunc{}
+
+type RatioFunc struct{}
+
+func (RatioFunc) Eval(ctx *Context, node *Node) ([]*types.PerfTrace, error) {
+ if len(node.Args) != 2 {
+ return nil, fmt.Errorf("ratio() takes two arguments")
+ }
+
+ tracesA, err := node.Args[0].Eval(ctx)
+ if err != nil {
+ return nil, fmt.Errorf("ratio() argument failed to evaluate: %s", err)
+ }
+
+ tracesB, err := node.Args[1].Eval(ctx)
+ if err != nil {
+ return nil, fmt.Errorf("ratio() argument failed to evaluate: %s", err)
+ }
+
+ ret := types.NewPerfTraceN(len(tracesA[0].Values))
+ for i, _ := range ret.Values {
+ ret.Values[i] = tracesA[0].Values[i] / tracesB[0].Values[i]
+ if math.IsInf(ret.Values[i], 0) {
+ ret.Values[i] = config.MISSING_DATA_SENTINEL
+ }
+ }
+ return []*types.PerfTrace{ret}, nil
+}
+
+func (RatioFunc) Describe() string {
+ return `ratio(a, b) returns the point by point ratio of two traces.
+ That is, it returns a trace with a[i]/b[i] for every point in a and b.`
+}
+
+var ratioFunc = RatioFunc{}
« no previous file with comments | « no previous file | perf/go/parser/parser.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698