JetRec: cleanup initialize/finalize messages
Cleanup the message logging and some other cleanups:
- remove finalize messages if they are not providing any useful information
- limit number of message during initialize
- reduce messaging (even if DEBUG) during execute
- take ToolHandle by reference in loops
This is mostly relevant for the trigger where we create a large number of instances of jet reconstruction algorithms.
Merge request reports
Activity
This merge request affects 1 package:
- Reconstruction/Jet/JetRec
This merge request affects 6 files:
- Reconstruction/Jet/JetRec/JetRec/JetAlgorithm.h
- Reconstruction/Jet/JetRec/JetRec/JetDecorationAlg.h
- Reconstruction/Jet/JetRec/JetRec/JetRecAlg.h
- Reconstruction/Jet/JetRec/src/JetAlgorithm.cxx
- Reconstruction/Jet/JetRec/src/JetDecorationAlg.cxx
- Reconstruction/Jet/JetRec/src/JetRecAlg.cxx
Adding @goetz as watcher
- Resolved by Frank Winklmeier
CI Result FAILURE (hash 83777930)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 46288]
added review-user-action-required label and removed review-pending-level-1 label
This merge request affects 1 package:
- Reconstruction/Jet/JetRec
This merge request affects 6 files:
- Reconstruction/Jet/JetRec/JetRec/JetAlgorithm.h
- Reconstruction/Jet/JetRec/JetRec/JetDecorationAlg.h
- Reconstruction/Jet/JetRec/JetRec/JetRecAlg.h
- Reconstruction/Jet/JetRec/src/JetAlgorithm.cxx
- Reconstruction/Jet/JetRec/src/JetDecorationAlg.cxx
- Reconstruction/Jet/JetRec/src/JetRecAlg.cxx
Adding @goetz as watcher
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash 83777930)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 46549]added review-approved label and removed review-pending-level-1 label
mentioned in commit a9a5e7de
added sweep:ignore label