Bug fixes for deadHVTool (#1708)

Co-authored-by: Michael Hank <[email protected]>
UCATLAS · Nov 13, 2024 · 26e575b · 26e575b
1 parent c38ee56
commit 26e575b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Root/ElectronSelector.cxx b/Root/ElectronSelector.cxx
@@ -405,11 +405,12 @@ EL::StatusCode ElectronSelector :: initialize ()
 
 
   // Set up the dead HV Removal Tool
-  if (m_applyDeadHVCellVeto)
+  if (m_applyDeadHVCellVeto) {
     m_deadHVTool.setTypeAndName("AsgDeadHVCellRemovalTool/deadHVTool");
     if (m_deadHVTool.retrieve().isFailure()){
       ANA_MSG_ERROR("Failed to retrieve DeadHVTool, aborting");
       return StatusCode::FAILURE;
+    }
   }
   else {
       ANA_MSG_WARNING("Not applying veto of dead HV cells although it's recommended - please double check!");

diff --git a/xAODAnaHelpers/ElectronSelector.h b/xAODAnaHelpers/ElectronSelector.h
@@ -286,7 +286,7 @@ class ElectronSelector : public xAH::Algorithm
   ElectronCutBasedPIDManager*              m_el_CutBased_PIDManager = nullptr;  //!
 
   /// @brief tool that selects on dead HV from the 2016 run, according to https://twiki.cern.ch/twiki/bin/view/AtlasProtected/EGammaIdentificationRun2#Removal_of_Electron_Photon_clust
-  asg::AnaToolHandle<IAsgDeadHVCellRemovalTool> m_deadHVTool;
+  asg::AnaToolHandle<IAsgDeadHVCellRemovalTool> m_deadHVTool; //!
 
   /* other private members */