Skip to content

Updated BremRecovery tracking flags

Thomas Strebler requested to merge tstreble/athena:BremConfigFlags into 23.0

This MR attempts to simplify the BremRecovery tracking flags. Since it has a minimal impact on trigger configs, this is targeted to 23.0.

Some part of the tracking reconstruction is configured to run the BremRecovery for all tracking passes (namely the TRT extension), while other parts only run it for the main tracking pass (the silicon TrackMaker and the ambiguity resolution). This structure is kept but a new TrackingPass flag doBremRecoverySi is introduced to avoid the extension check to be repeated in different places. Based on deeper investigations together with @christos, it should be safe to use the same config also for the TRT extension but this will be kept for master for safety.

The phiWidthBrem and etaWidthBrem flags are also set up globally now instead of per tracking pass, as the configuration of the CaloClusterROIPhiRZContainerMaker cannot be different for the various tracking passes and the same values were anyway used for all passes with brem recovery enabled.

FYI @christos @jmasik

Edited by Thomas Strebler

Merge request reports