tidy / formatting

Trigger/TrigSteer/TrigOutputHandling/src/DecisionSummaryMakerAlg.cxx

@@ -54,7 +54,8 @@ StatusCode DecisionSummaryMakerAlg::execute(const EventContext& context) const {
     }
     const auto thisCollFilter = m_collectionFilter.find( key.key() );
     if ( thisCollFilter == m_collectionFilter.end() ) {
-      ATH_MSG_WARNING( "The colleciton " << key.key() << " is not configured to contain any final decision, remove it from the configuration of " << name() << " to save time" );
+      ATH_MSG_WARNING( "The collection " << key.key() << " is not configured to contain any final decision," 
+                       << "remove it from the configuration of " << name() << " to save time" );
       continue;
     }
 
@@ -116,9 +117,9 @@ StatusCode DecisionSummaryMakerAlg::execute(const EventContext& context) const {
   decisionIDs( prescaledOutput ).insert( decisionIDs( prescaledOutput ).end(),
         prescaledIDs.begin(), prescaledIDs.end() ); // Save this to the output
 
-  // Save the set of chains which were flagged as only executing in rerun. This is a direct copy
+  // Save the set of chains which were flagged as only executing in rerun.
   DecisionIDContainer rerunIDs;
-  decisionIDs( rerunChains, rerunIDs ); // Extract from rerunChains (a Decision*) into rerunIDs (a set<int>)
+  decisionIDs( rerunChains, rerunIDs ); 
   decisionIDs( rerunOutput ).insert( decisionIDs( rerunOutput ).end(),
         rerunIDs.begin(), rerunIDs.end() );
 

Trigger/TrigSteer/TrigOutputHandling/src/DecisionSummaryMakerAlg.h

@@ -12,7 +12,9 @@
 
 /**
  * @class DecisionsSummaryMakerAlg
- * @brief makes one decision object containing only one decision ID per accepted chain
+ * @brief Executes after both the first-pass and secon-pass (rerun) chains have finished.
+ *        Makes decision objects containing decision IDs per passing chain, prescaled chain and rerun chain.
+ *        Sends end-of-HLT processing trigger to cost monitoring. Writes HLTSummary container.
  **/
 class DecisionSummaryMakerAlg : public AthReentrantAlgorithm {
 public:
@@ -26,18 +28,22 @@ public:
 private:
   SG::WriteHandleKey<TrigCompositeUtils::DecisionContainer> m_summaryKey{ this, "DecisionsSummaryKey", "HLTSummary", 
       "location of final decision" };
+
   SG::ReadHandleKeyArray<TrigCompositeUtils::DecisionContainer> m_finalDecisionKeys{ this, "FinalDecisionKeys", {}, 
       "Final stage of all decisions" };
+
   SG::WriteHandleKey<xAOD::TrigCompositeContainer> m_costWriteHandleKey { this, "CostWriteHandleKey", "TrigCostContainer",
     "Trig composite collections summarising the HLT execution" };
+
   SG::ReadHandleKey<xAOD::TrigCompositeContainer> m_l1SummaryKey { this, "L1DecoderSummaryKey", "L1DecoderSummary",
     "Chains status after L1 and prescaling" };
 
-
   ServiceHandle<ITrigCostMTSvc> m_trigCostSvcHandle { this, "TrigCostMTSvc", "TrigCostMTSvc", 
     "The trigger cost service" };
+
   Gaudi::Property< std::map< std::string, std::string > > m_lastStepForChain{ this, "FinalStepDecisions", {}, 
-      "The map of chain name to name of the collection in which the final decision is found" };
+    "The map of chain name to name of the collection in which the final decision is found" };
+
   Gaudi::Property<bool> m_enableCostMonitoring{this, "EnableCostMonitoring", false, 
     "Enables end-of-event cost monitoring behavior."};