Remove unused directory parameter

Author: PatKamin

devops/scripts/benchmarks/benches/base.py

@@ -49,8 +49,7 @@ class TracingType(Enum):
 
 
 class Benchmark(ABC):
-    def __init__(self, directory, suite):
-        self.directory = directory
+    def __init__(self, suite):
         self.suite = suite
 
     @abstractmethod
@@ -205,9 +204,9 @@ def run_bench(
 
     def create_data_path(self, name, skip_data_dir=False):
         if skip_data_dir:
-            data_path = os.path.join(self.directory, name)
+            data_path = os.path.join(options.workdir, name)
         else:
-            data_path = os.path.join(self.directory, "data", name)
+            data_path = os.path.join(options.workdir, "data", name)
             if options.redownload and Path(data_path).exists():
                 shutil.rmtree(data_path)
 

devops/scripts/benchmarks/benches/benchdnn.py

@@ -16,7 +16,7 @@
 
 
 class OneDnnBench(Suite):
-    def __init__(self, directory):
+    def __init__(self):
         self.project = None
 
     def git_url(self):

devops/scripts/benchmarks/benches/compute.py

@@ -43,7 +43,7 @@ def runtime_to_tag_name(runtime: RUNTIMES) -> str:
 
 
 class ComputeBench(Suite):
-    def __init__(self, directory):
+    def __init__(self):
         self.submit_graph_num_kernels = [4, 10, 32]
         self.project = None
 
@@ -301,7 +301,7 @@ class ComputeBenchmark(Benchmark):
     def __init__(
         self, bench, name, test, runtime: RUNTIMES = None, profiler_type: str = ""
     ):
-        super().__init__(options.workdir, bench)
+        super().__init__(bench)
         self.bench = bench
         self.bench_name = name
         self.test = test

devops/scripts/benchmarks/benches/gromacs.py

@@ -17,7 +17,7 @@
 
 
 class GromacsBench(Suite):
-    def __init__(self, directory):
+    def __init__(self):
         self.project = None
         model_path = str(Path(options.workdir) / self.grappa_file()).replace(
             ".tar.gz", ""

devops/scripts/benchmarks/benches/llamacpp.py

@@ -17,7 +17,7 @@
 
 
 class LlamaCppBench(Suite):
-    def __init__(self, directory):
+    def __init__(self):
         self.project = None
 
     def name(self) -> str:
@@ -73,7 +73,7 @@ def benchmarks(self) -> list[Benchmark]:
 
 class LlamaBench(Benchmark):
     def __init__(self, bench):
-        super().__init__(options.workdir, bench)
+        super().__init__(bench)
         self.bench = bench
 
     @property

devops/scripts/benchmarks/benches/syclbench.py

@@ -14,7 +14,7 @@
 
 
 class SyclBench(Suite):
-    def __init__(self, directory):
+    def __init__(self):
         self.project = None
 
     def name(self) -> str:
@@ -99,7 +99,7 @@ def benchmarks(self) -> list[Benchmark]:
 
 class SyclBenchmark(Benchmark):
     def __init__(self, bench, name, test):
-        super().__init__(options.workdir, bench)
+        super().__init__(bench)
         self.bench = bench
         self.bench_name = name
         self.test = test

devops/scripts/benchmarks/benches/test.py

@@ -58,7 +58,7 @@ def additional_metadata(self) -> dict[str, BenchmarkMetadata]:
 
 class TestBench(Benchmark):
     def __init__(self, suite, name, value, diff, group="", notes=None, unstable=None):
-        super().__init__("", suite)
+        super().__init__(suite)
         self.bname = name
         self.value = value
         self.diff = diff

devops/scripts/benchmarks/benches/umf.py

@@ -20,8 +20,7 @@ def isUMFAvailable():
 
 
 class UMFSuite(Suite):
-    def __init__(self, directory):
-        self.directory = directory
+    def __init__(self):
 
     def name(self) -> str:
         return "UMF"
@@ -44,7 +43,7 @@ def benchmarks(self) -> list[Benchmark]:
 
 class GBench(Benchmark):
     def __init__(self, bench):
-        super().__init__(bench.directory, bench)
+        super().__init__(bench)
 
         self.bench = bench
         self.bench_name = "umf-benchmark"

devops/scripts/benchmarks/benches/velocity.py

@@ -17,7 +17,7 @@
 
 
 class VelocityBench(Suite):
-    def __init__(self, directory) -> None:
+    def __init__(self) -> None:
         self.project = None
 
     def name(self) -> str:
@@ -57,7 +57,7 @@ def benchmarks(self) -> list[Benchmark]:
 
 class VelocityBase(Benchmark):
     def __init__(self, name: str, bin_name: str, vb: VelocityBench, unit: str):
-        super().__init__(options.workdir, vb)
+        super().__init__(vb)
         self.vb = vb
         self.bench_name = name
         self.bin_name = bin_name