Skip to content

Move HitDV chains to jFEX seeding [ATR-29954]

Teng Jian Khoo requested to merge khoo/athena:24.0-TrigLLPHypo-HitDVjFEX into 24.0

As @smh pointed out, we need the HitDV hypo to use jFEX jets rather than legacy L1 jets going forwards.

Here I switched from the legacy HLT_RecJetRoIs to the L1_jFexSRJetRoIs. I don't think there is a reason to use the RoIs from HLT seeding as opposed to straight from L1? This also simplifies reading the et/eta/phi. Hopefully I haven't missed some conversion factors somewhere, but tested with 200 events with no change between the legacy & Phase 1 versions. Verified that this removes the missing data dependency in @smh's test on ATR-29954.

Counts are the same as for legacy, so it appears the L1 jet seeding is fairly insensitive to which system supplies the seeds because the HLT jet preselection is quite tight -- meaning there is also room to tighten the cut on the seed if desired. We should not disable the legacy Jet Processor until we have this deployed at P1, otherwise the chains will break.

FYI @nagano, @mwessels, @thompson, @lidiaz, @gipezzul

Merge request reports