Skip to content

Attempt to fix ATLASRECTS-6778

Edward Moyse requested to merge emoyse/athena:master-fix-ATLASRECTS-6778 into master

This adds missing configuration of TrackToVertex / TrackCollectionCnvTool and thereby the Extrapolator (see ATLASRECTS-6778 for details of why this matters).

I'm sure this is required, but this line:

    trackCollectionCnvTool = acc.popToolsAndMerge( TrackCollectionCnvToolCfg(flags) )

reveals some old-style and CA mismatches i.e. there's a problem, but elsewhere.

Examples are:

RAWtoESD 09:07:43 AthenaConfiguration.ComponentAccumulator.ConfigurationError: Failed merging new config value (MuonDetectorManager) and old config value (ConditionStore+) for the (M
uonManagerKey) property of PixelDetectorElementCondAlg/PixelDetectorElementCondAlg (PixelDetectorElementCondAlg/PixelDetectorElementCondAlg) old (new).

and

RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    PrivateToolHandleArray HelperTools of EventSelectorByteStream/EventSelector does not have the same named components
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    Old (conf1) [] for EventSelectorByteStream/EventSelector
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    New (conf2) ['xAODMaker::EventInfoSelectorTool'] for EventSelectorByteStream/EventSelector
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    Will try to merge them, but this might go wrong!
RAWtoESD 09:07:42 Py:conf2toConfigurable    INFO ApplicationMgr property 'EvtSel' already set to 'EventSelector'.
RAWtoESD 09:07:42 Py:conf2toConfigurable    INFO ApplicationMgr property 'EvtSel' already set to 'EventSelector'.
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    PrivateToolHandleArray DetectorTools of GeoModelSvc/GeoModelSvc does not have the same named components
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    Old (conf1) ['AFP_GeoModelTool', 'BeamPipeDetectorTool', 'InDetServMatTool', 'LArDetectorToolNV', 'LUCID_DetectorTool', 'MuonDetec
torTool', 'PixelDetectorTool', 'SCT_DetectorTool', 'TRT_DetectorTool', 'TileDetectorTool', 'ZDC_DetTool'] for GeoModelSvc/GeoModelSvc
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    New (conf2) ['MuonDetectorTool'] for GeoModelSvc/GeoModelSvc
RAWtoESD 09:07:42 Py:conf2toConfigurable WARNING    Will try to merge them, but this might go wrong!

to my eye it seems like the CA configuration is misconfigured for older data (probably not surprisingly).

I'll keep this Draft until we fix it all, but I will start undrafted and run full-unit-tests , just to get more information.

Edited by Edward Moyse

Merge request reports