[tools][perfmon]: Bump CL delta thresholds
The timings report is too noisy.
Change-Id: Ie89811cf1541b2d07ade38ce52e3d423f80d7a65
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/179703
Kokoro: Kokoro <noreply+kokoro@google.com>
Commit-Queue: Ben Clayton <bclayton@google.com>
Reviewed-by: dan sinclair <dsinclair@chromium.org>
diff --git a/tools/src/cmd/perfmon/main.go b/tools/src/cmd/perfmon/main.go
index 7be08f6..26a46fc 100644
--- a/tools/src/cmd/perfmon/main.go
+++ b/tools/src/cmd/perfmon/main.go
@@ -1010,9 +1010,11 @@
})
if err != nil {
- body, _ := io.ReadAll(resp.Body)
info := &strings.Builder{}
- fmt.Fprintln(info, "response: ", string(body))
+ if resp.Body != nil {
+ body, _ := io.ReadAll(resp.Body)
+ fmt.Fprintln(info, "response: ", string(body))
+ }
fmt.Fprintln(info, "change-id: ", change.ChangeID)
fmt.Fprintln(info, "revision-id: ", currentHash.String())
fmt.Fprintln(info, "notify: ", notify)
@@ -1046,8 +1048,8 @@
return err
}
- const minDiff = time.Microsecond * 50 // Ignore time diffs less than this duration
- const minRelDiff = 0.01 // Ignore absolute relative diffs between [1, 1+x]
+ const minDiff = time.Millisecond // Ignore time diffs less than this duration
+ const minRelDiff = 0.05 // Ignore absolute relative diffs between [1, 1+x]
diff := bench.Compare(parentRun.Benchmarks, newRun.Benchmarks, minDiff, minRelDiff)
diffFmt := bench.DiffFormat{
TestName: true,
