Remove getTruthTauP4Prompt function from TauTruthMatchingTool - outdated
This MR is removing "getTruthTauP4Prompt" function from the TauTruthMatchingTool. These functions are trying to access elements which are not decorated anymore on the tau, so they are outdated (already outdated in R21)
Tagging @martindl
Merge request reports
Activity
This merge request affects 1 package:
- PhysicsAnalysis/TauID/TauAnalysisTools
This merge request affects 4 files:
- PhysicsAnalysis/TauID/TauAnalysisTools/Root/TauTruthMatchingTool.cxx
- PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/ITauTruthMatchingTool.h
- PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/TauTruthMatchingTool.h
- PhysicsAnalysis/TauID/TauAnalysisTools/doc/README-TauTruthMatchingTool.rst
Adding @martindl as watcher
added Analysis Tau analysis-review-required main review-pending-level-1 labels
CI Result FAILURE (hash 7ec83cba)Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3192]- Resolved by Vakhtang Tsulaia
Not sure the error in the CI is related to this MR
This merge request affects 1 package:
- PhysicsAnalysis/TauID/TauAnalysisTools
This merge request affects 4 files:
- PhysicsAnalysis/TauID/TauAnalysisTools/Root/TauTruthMatchingTool.cxx
- PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/ITauTruthMatchingTool.h
- PhysicsAnalysis/TauID/TauAnalysisTools/TauAnalysisTools/TauTruthMatchingTool.h
- PhysicsAnalysis/TauID/TauAnalysisTools/doc/README-TauTruthMatchingTool.rst
Adding @martindl as watcher
CI Result SUCCESS (hash 7ec83cba)Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3197]added review-approved label and removed review-pending-level-1 label
Approving since it only concerns removal of outdated code - AR
Edited by Romain Bouquetremoved analysis-review-required label
added analysis-review-approved label
mentioned in commit 55a41c3d