From 8575e02a36b2721645961f90e5bbcfeaaf43c860 Mon Sep 17 00:00:00 2001
From: Ioannis Magkanaris <iomagkanaris@gmail.com>
Date: Mon, 8 Apr 2024 15:30:25 +0200
Subject: [PATCH] Small bug fixes (#17)

- Fixed issue with regex when using `--label-regex` with `sbench-analyze plot`
- Removed deprecated `gpu_timers` parameter
---
 stencil_benchmarks/scripts/sbench_a100_collection.py  | 1 -
 stencil_benchmarks/scripts/sbench_analyze.py          | 6 +++---
 stencil_benchmarks/scripts/sbench_mi100_collection.py | 1 -
 stencil_benchmarks/scripts/sbench_mi50_collection.py  | 1 -
 stencil_benchmarks/scripts/sbench_v100_collection.py  | 1 -
 5 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/stencil_benchmarks/scripts/sbench_a100_collection.py b/stencil_benchmarks/scripts/sbench_a100_collection.py
index a5e9486..0c2631c 100644
--- a/stencil_benchmarks/scripts/sbench_a100_collection.py
+++ b/stencil_benchmarks/scripts/sbench_a100_collection.py
@@ -51,7 +51,6 @@ def main():
                                gpu_architecture='sm_80',
                                verify=False,
                                dry_runs=1,
-                               gpu_timers=True,
                                alignment=128,
                                dtype='float32')
 
diff --git a/stencil_benchmarks/scripts/sbench_analyze.py b/stencil_benchmarks/scripts/sbench_analyze.py
index 9c27f81..5d324db 100644
--- a/stencil_benchmarks/scripts/sbench_analyze.py
+++ b/stencil_benchmarks/scripts/sbench_analyze.py
@@ -191,10 +191,10 @@ def plot(csv, uniform, ylim, title, auto_group, group, select, filter,
 
     regexes = []
     for regex in label_regex:
-        splitter = label_regex[0]
-        if label_regex[-1] != splitter:
+        splitter = regex[0]
+        if regex[-1] != splitter:
             raise ValueError('expected input in the form /pattern/repl/')
-        pattern, repl = label_regex[1:-1].split(splitter, 1)
+        pattern, repl = regex[1:-1].split(splitter, 1)
         regexes.append((re.compile(pattern), repl))
 
     for index, row in df.iterrows():
diff --git a/stencil_benchmarks/scripts/sbench_mi100_collection.py b/stencil_benchmarks/scripts/sbench_mi100_collection.py
index 10de235..ca96336 100644
--- a/stencil_benchmarks/scripts/sbench_mi100_collection.py
+++ b/stencil_benchmarks/scripts/sbench_mi100_collection.py
@@ -51,7 +51,6 @@ def main():
                                gpu_architecture='gfx908',
                                verify=False,
                                dry_runs=100,
-                               gpu_timers=True,
                                alignment=512,
                                dtype='float32')
 
diff --git a/stencil_benchmarks/scripts/sbench_mi50_collection.py b/stencil_benchmarks/scripts/sbench_mi50_collection.py
index 5b94f8e..68ed22a 100644
--- a/stencil_benchmarks/scripts/sbench_mi50_collection.py
+++ b/stencil_benchmarks/scripts/sbench_mi50_collection.py
@@ -51,7 +51,6 @@ def main():
                                gpu_architecture='gfx906',
                                verify=False,
                                dry_runs=1,
-                               gpu_timers=True,
                                alignment=64,
                                dtype='float32')
 
diff --git a/stencil_benchmarks/scripts/sbench_v100_collection.py b/stencil_benchmarks/scripts/sbench_v100_collection.py
index 39cc700..1973545 100644
--- a/stencil_benchmarks/scripts/sbench_v100_collection.py
+++ b/stencil_benchmarks/scripts/sbench_v100_collection.py
@@ -51,7 +51,6 @@ def main():
                                gpu_architecture='sm_70',
                                verify=False,
                                dry_runs=1,
-                               gpu_timers=True,
                                alignment=128,
                                dtype='float32')