Skip to content

Commit 29246ba

Browse files
yn386dpgeorge
authored andcommitted
tests/run-perfbench.py: Fix issues when -s/-m is used with failed tests.
The option '-s' (--diff-score) or '-m' (--diff-time) fails when the specified result contains tests that was skipped or failed. This patch ignores "skipped: " or "failed: " message. Signed-off-by: Yuuki NAGAO <wf.yn386@gmail.com>
1 parent 2dc6873 commit 29246ba

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

tests/run-perfbench.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,13 @@ def parse_output(filename):
194194
m = int(m.split("=")[1])
195195
data = []
196196
for l in f:
197-
if ": " in l and ": SKIP" not in l and "CRASH: " not in l:
197+
if (
198+
": " in l
199+
and ": SKIP" not in l
200+
and "CRASH: " not in l
201+
and "skipped: " not in l
202+
and "failed: " not in l
203+
):
198204
name, values = l.strip().split(": ")
199205
values = tuple(float(v) for v in values.split())
200206
data.append((name,) + values)

0 commit comments

Comments
 (0)