Add clarification

dockstore · Feb 26, 2024 · cadb648 · cadb648
1 parent 648f988
commit cadb648
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...ggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java b/...ggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java
@@ -23,7 +23,8 @@ public List<TaskExecutions> getTaskExecutionsWithMetric(List<TaskExecutions> tas
     }
 
     /**
-     * Aggregates TaskExecutions that belong to a single workflow run into a workflow-level RunExecution. Does NOT check that the resulting workflow run is valid.
+     * Aggregates TaskExecutions that belong to a single workflow run into a workflow-level RunExecution. 
+     * Does NOT check that the resulting workflow run is valid. The validity check is done when workflow executions are aggregated so that the aggregator can recognize that task metrics were skipped.
      * @param taskExecutionsForOneWorkflowRun
      * @return
      */