Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 235
    • Merge requests 235
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !49765

Closed
Created Jan 19, 2022 by Edward Moyse@emoyseOwner
  • Report abuse
Report abuse

Attempt to fix ATLASRECTS-6778

  • Overview 16
  • Commits 1
  • Pipelines 2
  • Changes 2

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 Feb 15, 2022 by Edward Moyse
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master-fix-ATLASRECTS-6778