Skip to content

Remove RoI setting changes from Muon trigger reco sequence

Savanna Shaw requested to merge sshaw/athena:trigmuon-ca into master

Removing some cases where we were looping through the algorithms, and if there were RoI properties, resetting them. The RoI settings should all be set properly from the menu sequences (where the event views are defined). Changing them in the way this was done can have soem unintended consequences (for example, when running only the muon triggers, the RoI z-wdith for the LRT RoIs was being set to 10mm, which is not the desired behaviour. Luckily with a full menu, this didn't seem to adversely impact anything). Sine we then don't need to loop through all the algorithms, can also just add the algorithms directly to the reco sequence rather than creating a separate vector of algorithms first.

Merge request reports