From 1cf0365adcec716e3a8541f0eb78dbfab05a4a93 Mon Sep 17 00:00:00 2001 From: Simon Lee Date: Thu, 23 Apr 2026 13:22:46 +0000 Subject: [PATCH 1/2] fix: Fix metric keys for host-device benchmark --- Ironwood/src/benchmark_host_device.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Ironwood/src/benchmark_host_device.py b/Ironwood/src/benchmark_host_device.py index cc1e505e..6d47dab2 100644 --- a/Ironwood/src/benchmark_host_device.py +++ b/Ironwood/src/benchmark_host_device.py @@ -96,8 +96,8 @@ def benchmark_host_device( device_array.delete() return { - "H2D_Bandwidth_ms": h2d_perf, - "D2H_Bandwidth_ms": d2h_perf, + "h2d_bandwidth_ms": h2d_perf, + "d2h_bandwidth_ms": d2h_perf, } From c7e5c0b3ff97c091b4f5ae39fb796978f304b673 Mon Sep 17 00:00:00 2001 From: Simon Lee Date: Fri, 24 Apr 2026 02:27:24 +0000 Subject: [PATCH 2/2] fix: resolve line too long pylint errors --- Ironwood/src/benchmark_utils.py | 9 ++++++++- src/benchmark_utils.py | 9 ++++++++- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/Ironwood/src/benchmark_utils.py b/Ironwood/src/benchmark_utils.py index 9e0baa4e..0136012c 100644 --- a/Ironwood/src/benchmark_utils.py +++ b/Ironwood/src/benchmark_utils.py @@ -804,7 +804,14 @@ def upload_to_storage(trace_dir: str, local_file: str): if trace_dir.startswith("gs://"): # Google Cloud Storage (GCS) try: subprocess.run( - ["gcloud", "storage", "cp", "--recursive", local_file, trace_dir], + [ + "gcloud", + "storage", + "cp", + "--recursive", + local_file, + trace_dir, + ], check=True, capture_output=True, ) diff --git a/src/benchmark_utils.py b/src/benchmark_utils.py index a4ef714b..340836b6 100644 --- a/src/benchmark_utils.py +++ b/src/benchmark_utils.py @@ -243,7 +243,14 @@ def upload_to_storage(trace_dir: str, local_file: str): if trace_dir.startswith("gs://"): # Google Cloud Storage (GCS) try: subprocess.run( - ["gcloud", "storage", "cp", "--recursive", local_file, trace_dir], + [ + "gcloud", + "storage", + "cp", + "--recursive", + local_file, + trace_dir, + ], check=True, capture_output=True, )