Skip to content
Snippets Groups Projects

Configure the ACTS geometry tool in legacy trigger via wrapped CA [ATR-27768]

Closed Teng Jian Khoo requested to merge khoo/athena:23.0-InDetTrigVertices-ActsGeoFromCA into 23.0

While working on !61235 (merged), I encountered a clash between a statically defined tool configuration returned by ConfiguredActsTrackingGeometry (building Pixel only) and a full configuration originating from the CA-based HLT jet tracking instance:

AthenaConfiguration.ComponentAccumulator.ConfigurationError: Failed merging new config value (['Pixel', 'SCT']) and old config value (['Pixel']) for the (BuildSubDetectors) property of ActsTrackingGeometrySvc/ActsTrackingGeometrySvc (ActsTrackingGeometrySvc/ActsTrackingGeometrySvc) old (new).

This could be resolved by replacing the ConfiguredActsTrackingGeometry call with the proper ActsTrackingGeometryToolCfg taking the tracking instance ConfigFlags. It might be possible to remove this function and the containing file ​ActsTrackingGeometryTool.py entirely, as it was only called from this one place.

Note: labelled sweep:ignore because the ACTS packages/files have been renamed in master.

FYI @sutt

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading