From bce536b9b4dcc33c8166b6cfd79712326ab79fe1 Mon Sep 17 00:00:00 2001
From: Christoph Kluge <christoph.kluge@fau.de>
Date: Fri, 20 Dec 2024 17:15:02 +0100
Subject: [PATCH] fix: add missing parameters for correct shared metric
 thresholds

---
 web/frontend/src/generic/plots/MetricPlot.svelte | 8 ++++----
 web/frontend/src/job/Metric.svelte               | 4 ++++
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/web/frontend/src/generic/plots/MetricPlot.svelte b/web/frontend/src/generic/plots/MetricPlot.svelte
index f08c86d..c42c861 100644
--- a/web/frontend/src/generic/plots/MetricPlot.svelte
+++ b/web/frontend/src/generic/plots/MetricPlot.svelte
@@ -50,7 +50,7 @@
   }
 
   // removed arg "subcluster": input metricconfig and topology now directly derived from subcluster
-  function findThresholds(
+  function findJobAggregationThresholds(
     subClusterTopology,
     metricConfig,
     scope,
@@ -60,7 +60,7 @@
   ) {
 
     if (!subClusterTopology || !metricConfig || !scope) {
-      console.warn("Argument missing for findThresholds!");
+      console.warn("Argument missing for findJobAggregationThresholds!");
       return null;
     }
 
@@ -96,7 +96,7 @@
       else if (scope == "hwthread")     divisor = subClusterTopology.core.length; // alt. name for core
       else if (scope == "accelerator")  divisor = subClusterTopology.accelerators.length;
       else {
-        console.log('Unknown scope, return default thresholds ', scope)
+        console.log('Unknown scope, return default aggregation thresholds ', scope)
         divisor = 1;
       }
 
@@ -157,7 +157,7 @@
     caution: "rgba(255, 128, 0, 0.3)",
     alert: "rgba(255, 0, 0, 0.3)",
   };
-  const thresholds = findThresholds(
+  const thresholds = findJobAggregationThresholds(
     subClusterTopology,
     metricConfig,
     scope,
diff --git a/web/frontend/src/job/Metric.svelte b/web/frontend/src/job/Metric.svelte
index 2e3967d..b61ed13 100644
--- a/web/frontend/src/job/Metric.svelte
+++ b/web/frontend/src/job/Metric.svelte
@@ -213,6 +213,8 @@
       timestep={data.timestep}
       scope={selectedScope}
       metric={metricName}
+      numaccs={job.numAcc}
+      numhwthreads={job.numHWThreads}
       {series}
       {isShared}
       {zoomState}
@@ -226,6 +228,8 @@
       timestep={data.timestep}
       scope={selectedScope}
       metric={metricName}
+      numaccs={job.numAcc}
+      numhwthreads={job.numHWThreads}
       {series}
       {isShared}
       {zoomState}