fix misleading warning about CaloClusterContainer/CombinedTower
In a recent Tier0 log, I found the warning
SGInputLoader 88 1 WARNING unable to find proxy for ( 'xAOD::CaloClusterContainer' , 'StoreGateSvc+CombinedTower' )
Re-running with AvalancheSchedulerSvc.ShowDataDependencies=True
, I learned that the HIEventShapeMaker
algorithm fails to retrieve this container, which I found rather worrisome. Looking at the code, it turns out that the HIEventShapeMaker actually retrieves the correct CombinedTowerContainer via it's base-class INavigable4MomentumCollection
. It can work also with a xAOD::CaloClusterContainer
as input but in the current configuration it uses a CombinedTowerContainer
(though the INavigable4MomentumCollection interface). The warning is rather the symptom of a mis-configuration that is fixed with this MR.
In the absence of a CI for HI, I manually validated this change on 100 events of data23_hi.00462205.express_express.merge.RAW._lb1242._SFO-ALL._0001.1
cc @stapiaar
Merge request reports
Activity
This merge request affects 1 package:
- Reconstruction/HeavyIonRec/HIGlobal
This merge request affects 2 files:
- Reconstruction/HeavyIonRec/HIGlobal/python/HIGlobalConfig.py
- Reconstruction/HeavyIonRec/HIGlobal/src/HIEventShapeMaker.cxx
Adding @goetz as watcher
added 23.0 JetEtmiss Reconstruction review-pending-level-1 labels
The system selected 50 tests to probe the Athena changeset (out of 70 available tests). Link to tests selection rules CI Result SUCCESS (hash f3d5c281)Athena 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 79024]removed review-pending-level-1 label
added review-approved label
added review-approved-tier0 label
mentioned in commit 48d8f60d
mentioned in merge request !66339 (merged)
mentioned in merge request !66421 (merged)