Skip to content

Change HLT seeding from L1J to use L1Calo RoIs instead of FS

Rafal Bielski requested to merge rbielski/athena:fix-l1j-hltseeding into master

Any chain based on (legacy) L1Calo Jet thresholds with explicitly set l1SeedThresholds=['JX'] (X=12,15,...) should be using L1Calo Jet RoI as its initialRoI. Chains which are the subject of ATR-24753 are an example (L1J-seeded RoI-based PEB streamers).

Fix the HLTSeeding implementation such that the L1J thresholds unpacking tool follows the same logic as L1EM and creates the correct initialRoI collection.

Note that almost all chains in the menu based on J items are explicitly configured with l1SeedThresholds=['FSNOSEED'], and such setting is even enforced for Jet/MET/bJet signatures. These chains are not affected by this MR. Only the chains from ATR-24753 are affected, plus the chain HLT_larpsall_L1J15 which has been incorrectly using a full-scan RoI up to know, but should've been using the L1Calo Jet RoI. The counts of that chain change due to this fix.

Edited by Rafal Bielski

Merge request reports