For len(failed_quanta)>5, display counts and not data_ids

lsst · Apr 26, 2024 · 3c3e9d6 · 3c3e9d6
1 parent 9f21ca1
commit 3c3e9d6
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 9 deletions.
diff --git a/python/lsst/ctrl/mpexec/cli/script/report.py b/python/lsst/ctrl/mpexec/cli/script/report.py
@@ -79,14 +79,22 @@ def report(
                 dataset_table_rows.append(summary_dict[task]["outputs"][data_product])
                 data_products.append(data_product)
 
-            quanta_summary.append(
-                {
-                    "Task": task,
-                    "Failed Quanta": summary_dict[task]["failed_quanta"],
-                    "Blocked Quanta": summary_dict[task]["n_quanta_blocked"],
-                }
-            )
-
+            if len(summary_dict[task]["failed_quanta"]) > 5:
+                quanta_summary.append(
+                    {
+                        "Task": task,
+                        "Failed Quanta": len(summary_dict[task]["failed_quanta"]),
+                        "Blocked Quanta": summary_dict[task]["n_quanta_blocked"],
+                    }
+                )
+            else:
+                quanta_summary.append(
+                    {
+                        "Task": task,
+                        "Failed Quanta": summary_dict[task]["failed_quanta"],
+                        "Blocked Quanta": summary_dict[task]["n_quanta_blocked"],
+                    }
+                )
             if "errors" in summary_dict[task].keys():
                 error_summary.append({task: summary_dict[task]["errors"]})
         quanta = Table(quanta_summary)

diff --git a/tests/test_cliCmdReport.py b/tests/test_cliCmdReport.py
@@ -98,7 +98,6 @@ def test_report(self):
         # Check that task0 and the failed quanta for task0 exist in the string
         self.assertIn("task0", result_hr.stdout)
         self.assertIn("Failed Quanta", result_hr.stdout)
-        self.assertIn("{'data_id': {'instrument': 'INSTR', 'detector': 0}}", result_hr.stdout)
 
 
 if __name__ == "__main__":