Fixed the access to CaloDetDescrManager in TBRec (part I)
- Switched
TBCellContainerFillerTool
to readingCaloDetDescrManager
from the Condition Store - Dropped the redundant dependencies on
CaloDetDescrManager
fromTBClusterMaker
,TBECLArRawChannelBuilder
andTBEMECXTalkToyModel
. Instead they now retrieveCaloCell
ID helpers directly from the Detector Store
This is part of the ongoing migration of all clients of CaloDetDescrManager
Merge request reports
Activity
This merge request affects 1 package:
- TestBeam/TBRec
This merge request affects 8 files:
- TestBeam/TBRec/src/TBCellContainerFillerTool.cxx
- TestBeam/TBRec/src/TBCellContainerFillerTool.h
- TestBeam/TBRec/src/TBClusterMaker.cxx
- TestBeam/TBRec/src/TBClusterMaker.h
- TestBeam/TBRec/src/TBECLArRawChannelBuilder.cxx
- TestBeam/TBRec/src/TBECLArRawChannelBuilder.h
- TestBeam/TBRec/src/TBEMECXTalkToyModel.cxx
- TestBeam/TBRec/src/TBEMECXTalkToyModel.h
added TestBeam master review-pending-level-1 labels
CI Result SUCCESS (hash 3f2669f9)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view
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 41203]added review-approved label and removed review-pending-level-1 label
mentioned in commit d5391910
added sweep:ignore label