Skip to content

TrigT2CaloCommon: always use Ringer in fastCaloRecoSequence

Frank Winklmeier requested to merge fwinkl/athena:doringer into master

Remove the option to enable/disable the Ringer in fastCaloRecoSequence because in any case only one instance (with a unique name) of T2CaloEgammaReFastAlgo is used. This means that even if fastCaloRecoSequence is called once with and once without Ringer, the first one wins. We were lucky that the physics sequences always got configured first (with Ringer enabled) and then the setup of the calibration sequence (without Ringer) just re-used the existing algorithm instance.

I noticed this bug when trying to migrate some of this code to CA, because there we always get unique algorithm instances even if they have the same name. In practice, this change has no impact as we are always using the Ringer these days.

Merge request reports