From b09da3577b49febd77f1e723d4c929a127136d98 Mon Sep 17 00:00:00 2001
From: Federica Piazza <federica.piazza@cern.ch>
Date: Fri, 21 Mar 2025 10:16:52 -0500
Subject: [PATCH] -a

---
 .../src/OfflineObjectDecorHelper.cxx                | 13 -------------
 .../src/OfflineObjectDecorHelper.h                  |  5 -----
 .../InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx | 10 ++++------
 3 files changed, 4 insertions(+), 24 deletions(-)

diff --git a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.cxx b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.cxx
index 0995303cfbb9..5e8fff1d4585 100644
--- a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.cxx
+++ b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.cxx
@@ -68,19 +68,6 @@ namespace IDTPM {
         track, "truthParticleLink" );
   }
 
-  /// isFake
-  bool isFakeTruth( const xAOD::TrackParticle& track,
-                    const float truthProbCut,
-                    const bool unlinkedAsFakes )
-  {
-    /// if fakes include unlinked, return true if isUnlinked is true
-    if( unlinkedAsFakes and isUnlinkedTruth( track ) ) return true;
-    float prob = getTruthMatchProb( track );
-    /// returns true if truthMatchProbability deco isn't available or
-    /// if the truth matching probability is below theshold
-    return ( prob < truthProbCut );
-  }
-
   /// isReconstructable
   bool isReconstructable( const xAOD::TruthParticle& truth,
                           const std::vector<unsigned int>& minSilHits,
diff --git a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.h b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.h
index fc5b20685861..98e3f311312f 100644
--- a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.h
+++ b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/OfflineObjectDecorHelper.h
@@ -82,11 +82,6 @@ namespace IDTPM {
   inline const xAOD::TruthParticle* getLinkedTruth(
       const xAOD::TruthParticle&, const float ) { return nullptr; }; // dummy - to avoid compilation errors
 
-  bool isFakeTruth( const xAOD::TrackParticle& track,
-                    const float truthProbCut=0., const bool unlinkedAsFakes=false );
-  inline bool isFakeTruth( const xAOD::TruthParticle&,
-                           const float, const bool ) { return false; }; // dummy - to avoid compilation errors
-
   bool isReconstructable( const xAOD::TruthParticle& truth,
                           const std::vector< unsigned int >& minSilHits,
                           const std::vector< float >& etaBins);
diff --git a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx
index 402c978dd0fc..a9b64b233436 100644
--- a/InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx
+++ b/InnerDetector/InDetValidation/InDetTrackPerfMon/src/TrackAnalysisPlotsMgr.cxx
@@ -366,10 +366,10 @@ StatusCode IDTPM::TrackAnalysisPlotsMgr::fillPlotsTest(
 
     /// fake rate plots (and hits on fake plots)
     bool isUnlinked = isUnlinkedTruth( *particle );
-    bool notTruthMatched = getTruthMatchProb( *particle ) < 0.5;
+    bool isFakeTruth = getTruthMatchProb( *particle ) < m_trkAnaDefSvc->truthProbCut();
     if ( isUnlinked ) {
       if( m_plots_missingTruth ) {
-        ATH_CHECK( m_plots_missingTruth->fillPlots( *particle, notTruthMatched, truthMu, actualMu, weight ) );
+        ATH_CHECK( m_plots_missingTruth->fillPlots( *particle, isFakeTruth, truthMu, actualMu, weight ) );
       }
       if( m_plots_hitsOnUnlinkedTrk ) {
         ATH_CHECK( m_plots_hitsOnUnlinkedTrk->fillPlots( *particle, truthMu, actualMu, weight ) ); 
@@ -378,10 +378,8 @@ StatusCode IDTPM::TrackAnalysisPlotsMgr::fillPlotsTest(
 
     bool doFakes = m_trkAnaDefSvc->unlinkedAsFakes() ? true : not isUnlinked;
     if( doFakes and m_plots_fakeRate ) {
-      bool isFake = isFakeTruth( *particle, m_trkAnaDefSvc->truthProbCut(),
-                                 m_trkAnaDefSvc->unlinkedAsFakes() );
-      ATH_CHECK( m_plots_fakeRate->fillPlots( *particle, isFake, truthMu, actualMu, weight ) );
-      if( m_plots_hitsOnFakeTrk and isFake ) {
+      ATH_CHECK( m_plots_fakeRate->fillPlots( *particle, isFakeTruth, truthMu, actualMu, weight ) );
+      if( m_plots_hitsOnFakeTrk and isFakeTruth ) {
         ATH_CHECK( m_plots_hitsOnFakeTrk->fillPlots( *particle, truthMu, actualMu, weight ) );
       }
     }
-- 
GitLab