appease monitoring build

AlexWilkinsonnn · AndyChappell · commit 6fb1f820eb5b · 2026-03-02T10:31:09.000+01:00
diff --git a/larpandoracontent/LArMonitoring/VisualMonitoringAlgorithm.cc b/larpandoracontent/LArMonitoring/VisualMonitoringAlgorithm.cc
@@ -218,14 +218,15 @@ void VisualMonitoringAlgorithm::VisualizeCaloHitList(const std::string &listName
         volIdToCaloHits[volId].push_back(pCaloHit);
     }
 
-    int colorIter{static_cast<int>(RED)};
+    [[maybe_unused]] int colorIter{0};
     for (const auto &[volId, caloHitListPartition] : volIdToCaloHits)
     {
         const std::string label{(listName.empty() ? "CurrentCaloHits" : listName) +
             ", volId ("  + std::to_string(volId.first) + ", " + std::to_string(volId.second) + ")"};
-        PANDORA_MONITORING_API(VisualizeCaloHits(
-            this->GetPandora(), &caloHitListPartition, label.c_str(), static_cast<Color>(colorIter++)));
-        colorIter = static_cast<Color>(colorIter) == AUTO ? static_cast<int>(RED) : colorIter;
+        PANDORA_MONITORING_API(VisualizeCaloHits(this->GetPandora(), &caloHitListPartition, label.c_str(),
+            static_cast<Color>(++colorIter) < AUTO          ? static_cast<Color>(colorIter)
+                : (colorIter % static_cast<int>(AUTO)) == 0 ? static_cast<Color>(++colorIter % static_cast<int>(AUTO))
+                                                            : static_cast<Color>(colorIter % static_cast<int>(AUTO))));
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -218,14 +218,15 @@ void VisualMonitoringAlgorithm::VisualizeCaloHitList(const std::string &listName`
`218`	`218`	`volIdToCaloHits[volId].push_back(pCaloHit);`
`219`	`219`	`}`
`220`	`220`
`221`		`- int colorIter{static_cast<int>(RED)};`
	`221`	`+ [[maybe_unused]] int colorIter{0};`
`222`	`222`	`for (const auto &[volId, caloHitListPartition] : volIdToCaloHits)`
`223`	`223`	`{`
`224`	`224`	`const std::string label{(listName.empty() ? "CurrentCaloHits" : listName) +`
`225`	`225`	`", volId (" + std::to_string(volId.first) + ", " + std::to_string(volId.second) + ")"};`
`226`		`- PANDORA_MONITORING_API(VisualizeCaloHits(`
`227`		`- this->GetPandora(), &caloHitListPartition, label.c_str(), static_cast<Color>(colorIter++)));`
`228`		`- colorIter = static_cast<Color>(colorIter) == AUTO ? static_cast<int>(RED) : colorIter;`
	`226`	`+ PANDORA_MONITORING_API(VisualizeCaloHits(this->GetPandora(), &caloHitListPartition, label.c_str(),`
	`227`	`+ static_cast<Color>(++colorIter) < AUTO ? static_cast<Color>(colorIter)`
	`228`	`+ : (colorIter % static_cast<int>(AUTO)) == 0 ? static_cast<Color>(++colorIter % static_cast<int>(AUTO))`
	`229`	`+ : static_cast<Color>(colorIter % static_cast<int>(AUTO))));`
`229`	`230`	`}`
`230`	`231`	`}`
`231`	`232`