ATR-27270: jet/b-jet trigger physics menu downshift
This adds new chains for which the jet trigger threshold is moved down by about 6% for almost every lowest unprescaled primary jet and b-jet trigger for Physics_pp_Run3
menu. This in preparation of a small-R PFlow jet calibration update at HLT, which improves the HLT jet pT response and resolution but jets incur a ~6% downwards shift in response (HLT jets formerly overcorrected).
Affects trigger counts: Expect higher trigger rate from new chains with current HLT jet calibration.
This was done via a python script by Dan has in gitlab: it just adds another otherwise identical menu item with a #downshift
comment appended. Pinging @khoo, @cantel, @lbeemste, @lshi, @cpollard.
Duplicate of !62353 (closed), from a common branch. @lbeemste will have to re-add emojis.
Merge request reports
Activity
mentioned in merge request !62353 (closed)
added 23.0 Trigger TriggerMenu labels
CI Result FAILURE (hash 6f2bfc7d)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 69841]added 2 commits
added urgent label
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
added changes-trigger-counts review-pending-level-1 labels
WARNING: big files (>100K) are found in the changeset 240K in file Trigger/TrigValidation/TrigP1Test/share/ref_v1Dev_decodeBS_build.ref 412K in file Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref 400K in file Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Menu/Physics_pp_run3_v1.pyCI failed, please have a look, changing tag to review-user-action-required
removed review-pending-level-1 label
added review-user-action-required label
@joany please wait for the new CI, the result there is from before the MR was undrafted.
removed review-user-action-required label
added review-pending-level-1 label
@fgiuli FYI there may be some express stream changes here due to the threshold updates.
CI Result SUCCESS (hash 1e4cad44)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 69910]- Resolved by Lidija Zivkovic
CI passed changing the flag to review-approved is this OK from the physics point of view?
Edited by Joany Manjarres
removed review-pending-level-1 label
added review-approved label
added review-approved-point1 label
added review-approved-tier0 label
mentioned in commit 74fb1d53
mentioned in merge request !62427 (merged)
mentioned in merge request !62475 (merged)