From a838b0c07be6b86d695d9e967c06043ee47a1549 Mon Sep 17 00:00:00 2001 From: vedithal-amd Date: Thu, 29 Jan 2026 18:54:52 -0500 Subject: [PATCH] [rocprofiler-compute] Fix test case for MI 308 (#2934) * Fix test case for MI 308 * Use consistent naming of GPUs in comment --- .../tests/test_metric_validation.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/projects/rocprofiler-compute/tests/test_metric_validation.py b/projects/rocprofiler-compute/tests/test_metric_validation.py index 56c01d236f..eeff51d40b 100644 --- a/projects/rocprofiler-compute/tests/test_metric_validation.py +++ b/projects/rocprofiler-compute/tests/test_metric_validation.py @@ -60,11 +60,15 @@ VALIDATE_METRICS = { "metric_id": "4.1.9", "csv_file": "4.1_Roofline_Performance_Rates.csv", "column": "Value", - # MI 300 series contains MI325X GPU which + # MI 300 series contains MI 325X GPU which # uses improved HBM3E instead of HBM3 used in - # MI300X GPU. Hence, multiple expected values + # MI 300X GPU. Hence, multiple expected values # to cover both cases. - "expected_values": [3910.62, 4287.31], + # MI 308 has lower bandwidth. + # MI 300X: 3910.62 GB/s + # MI 325X: 4287.31 GB/s + # MI 308: 2003.45 GB/s + "expected_values": [2003.45, 3910.62, 4287.31], }, ], "MI350": [