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

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: add some traces to the test 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_test.go » ('j') | perf/go/parser/parser_test.go » ('J')
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..7043be46ca498ce8fc9cb3c20089479cc77d847f 100644
--- a/perf/go/parser/funcs.go
+++ b/perf/go/parser/funcs.go
@@ -175,3 +175,35 @@ 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")
+ }
+
+ traces, err := node.Args[0].Eval(ctx)
+ if err != nil {
+ return nil, fmt.Errorf("ratio() argument failed to evaluate: %s", err)
+ }
+
+ traces, err = node.Args[1].Eval(ctx)
+ if err != nil {
+ return nil, fmt.Errorf("ratio() argument failed to evaluate: %s", err)
+ }
+
+ ret := types.NewPerfTraceN(len(traces[0].Values))
+ for i, _ := range ret.Values {
+ fmt.Println(traces[0].Values[i])
tfarina 2014/10/16 17:42:30 ret.Values[i] = traces[0].Values[i] / traces[1].Va
+ }
+
+ return nil, nil
tfarina 2014/10/16 17:42:30 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_test.go » ('j') | perf/go/parser/parser_test.go » ('J')

Powered by Google App Engine
This is Rietveld 408576698