CaloRecGPU: migrated to reading CaloDetDescrManager from the CondStore
BasicConstantGPUDataExporter
switched to reading CaloDDManager
from the CondStore
.
Other classes changed by this commit switched to retrieving CaloCell_ID
helper
from the Detector Store instead of getting it through the Calo DD manager.
CaloDetDescrManager::instance()
will soon be deprecated
Merge request reports
Activity
This merge request affects 1 package:
- Calorimeter/CaloRecGPU
This merge request affects 10 files:
- Calorimeter/CaloRecGPU/src/BasicConstantGPUDataExporter.cxx
- Calorimeter/CaloRecGPU/src/BasicConstantGPUDataExporter.h
- Calorimeter/CaloRecGPU/src/BasicEventDataGPUExporter.cxx
- Calorimeter/CaloRecGPU/src/BasicEventDataGPUExporter.h
- Calorimeter/CaloRecGPU/src/BasicGPUToAthenaImporter.cxx
- Calorimeter/CaloRecGPU/src/BasicGPUToAthenaImporter.h
- Calorimeter/CaloRecGPU/src/CaloCPUOutput.cxx
- Calorimeter/CaloRecGPU/src/CaloCPUOutput.h
- Calorimeter/CaloRecGPU/src/CaloCellsCounterCPU.cxx
- Calorimeter/CaloRecGPU/src/CaloCellsCounterCPU.h
Adding @pavol as watcher
added Calorimeter JetEtmiss master review-pending-level-1 labels
FYI @dossantn I have not tested my changes at runtime. If you could run some tests to make sure I did not break anything, it would be appreciated.
CI Result SUCCESS (hash 50fa1330)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon 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
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 59338]added review-approved label and removed review-pending-level-1 label
@tsulaia, I have managed to run a few tests and the results seem to be the same as before, so I'd say everything appears to be working perfectly. My apologies for the delay.
mentioned in commit b6e0e234
added sweep:ignore label