Skip to content

TriggerMenuMT: Align PEB steps

Rafal Bielski requested to merge rbielski/athena:align-peb-steps into master

If multiple chains use the same event building sequence (e.g. LArPEB like in !35672 (merged)) then the PEB sequence has to be in the same step for all these chains. Otherwise, this leads to a stall.

Implement adding empty steps before PEB step where needed to align the PEB sequences. Largely based on work done by @hrussell, but independent of the signature alignment, which happens afterwards. Whether these two alignments interfere remains to be seen.

Example effect (tested by running with this MR + !35672 (merged)):

 Built CF for chain HLT_g20_etcut_LArPEB_L1EM15 with 5 steps: 
   - {'Step1_1PhotonFastCalo': [1]}
   - {'Step2_1FastPhoton': [1]}
   - {'Step3_1PhotonPrecisionCalo': [1]}
   - {'Step4_1PhotonPrecisionCalo_EmptyPEBAlign1_Step4_PEBInfoWriter_LArPEB': []}
   - {'Step5_PEBInfoWriter_LArPEB': [1]}

Closes ATR-22012 and unblocks !35672 (merged)

Merge request reports