From 128cc46c032b9bb24e1bfb8cb87d79b625792167 Mon Sep 17 00:00:00 2001
From: Victor Hugo Ruelas Rivera <victor.hugo.ruelas.rivera@cern.ch>
Date: Mon, 21 Nov 2022 12:40:19 +0100
Subject: [PATCH] Substitute period with p for run2 trigger decos

---
 .../Root/TrigEventSelectorAlg.cxx             | 34 +++++++++++--------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/PhysicsAnalysis/Algorithms/TriggerAnalysisAlgorithms/Root/TrigEventSelectorAlg.cxx b/PhysicsAnalysis/Algorithms/TriggerAnalysisAlgorithms/Root/TrigEventSelectorAlg.cxx
index 7b04dbdf75f2..b1d8770baf2b 100644
--- a/PhysicsAnalysis/Algorithms/TriggerAnalysisAlgorithms/Root/TrigEventSelectorAlg.cxx
+++ b/PhysicsAnalysis/Algorithms/TriggerAnalysisAlgorithms/Root/TrigEventSelectorAlg.cxx
@@ -10,9 +10,9 @@
 #include <xAODEventInfo/EventInfo.h>
 
 CP::TrigEventSelectionAlg::TrigEventSelectionAlg(const std::string &name,
-                                             ISvcLocator *svcLoc)
-  : EL::AnaAlgorithm(name, svcLoc),
-    m_trigDecisionTool("Trig::TrigDecisionTool/TrigDecisionTool")
+                                                 ISvcLocator *svcLoc)
+    : EL::AnaAlgorithm(name, svcLoc),
+      m_trigDecisionTool("Trig::TrigDecisionTool/TrigDecisionTool")
 {
   declareProperty("tool", m_trigDecisionTool, "trigger decision tool");
   declareProperty("triggers", m_trigList, "trigger selection list");
@@ -21,29 +21,33 @@ CP::TrigEventSelectionAlg::TrigEventSelectionAlg(const std::string &name,
 
 StatusCode CP::TrigEventSelectionAlg::initialize()
 {
-  if (m_trigList.empty()) {
+  if (m_trigList.empty())
+  {
     ATH_MSG_ERROR("A list of triggers needs to be provided");
     return StatusCode::FAILURE;
   }
 
   ANA_CHECK(m_trigDecisionTool.retrieve());
 
-  if (!m_selectionDecoration.empty()) {
-    for (const std::string &chain : m_trigList) {
-      m_selectionAccessors.emplace_back(m_selectionDecoration + "_" + RCU::substitute (chain, "-", "_"));
+  if (!m_selectionDecoration.empty())
+  {
+    for (const std::string &chain : m_trigList)
+    {
+      m_selectionAccessors.emplace_back(m_selectionDecoration + "_" + RCU::substitute(RCU::substitute(chain, ".", "p"), "-", "_"));
     }
   }
 
-  ANA_CHECK (m_filterParams.initialize());
+  ANA_CHECK(m_filterParams.initialize());
 
   return StatusCode::SUCCESS;
 }
 
 StatusCode CP::TrigEventSelectionAlg::execute()
 {
-  FilterReporter filter (m_filterParams, m_noFilter.value());
+  FilterReporter filter(m_filterParams, m_noFilter.value());
 
-  if (m_trigList.empty()) {
+  if (m_trigList.empty())
+  {
     filter.setPassed(true);
     return StatusCode::SUCCESS;
   }
@@ -51,13 +55,15 @@ StatusCode CP::TrigEventSelectionAlg::execute()
   const xAOD::EventInfo *evtInfo = 0;
   ANA_CHECK(evtStore()->retrieve(evtInfo, "EventInfo"));
 
-  for (size_t i = 0; i < m_trigList.size(); i++) {
+  for (size_t i = 0; i < m_trigList.size(); i++)
+  {
     bool trigPassed = m_trigDecisionTool->isPassed(m_trigList[i]);
-    if (!m_selectionDecoration.empty()) {
+    if (!m_selectionDecoration.empty())
+    {
       m_selectionAccessors[i](*evtInfo) = trigPassed;
     }
     if (trigPassed)
-      filter.setPassed (true);
+      filter.setPassed(true);
   }
 
   return StatusCode::SUCCESS;
@@ -65,7 +71,7 @@ StatusCode CP::TrigEventSelectionAlg::execute()
 
 StatusCode CP::TrigEventSelectionAlg::finalize()
 {
-  ANA_MSG_INFO (m_filterParams.summary());
+  ANA_MSG_INFO(m_filterParams.summary());
 
   return StatusCode::SUCCESS;
 }
-- 
GitLab