Update JetEffMon algorithm to remove unnecessary use of warnings

Update jeteffmon algorithm to remove the unnecessary use of warnings (replaced with debug) Also removed some unused list items and updated LR gFEX TOB threshold finder. Previously, the threshold finder assumed that the threshold is the number in the trigger name, but for the large radius thresholds, this is actually wrong (gJ80 actually corresponds to a cut of 50 Gev, gJ100 is 70 and so on), this 30 GeV subtraction has been implemented in the threshold finder now

assigned to @will, @atlasbot, and @krdona

This merge request affects 1 package:

• Trigger/TrigT1/TrigT1CaloMonitoring

This merge request affects 2 files:

• Trigger/TrigT1/TrigT1CaloMonitoring/python/JetEfficiencyMonitorAlgorithm.py
• Trigger/TrigT1/TrigT1CaloMonitoring/src/JetEfficiencyMonitorAlgorithm.cxx

Adding @will ,@thompson ,@jlittle as watchers

added 24.0 DQ JetEtmiss L1Calo Trigger review-pending-level-1 labels

CI Result SUCCESS (hash 03e8f302)

	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 (remote access info)

Changes are fine and CI succeeds, approving for L1. -l1

added review-approved label and removed review-pending-level-1 label

enabled an automatic merge when all merge checks for 03e8f302 pass

canceled the automatic merge

@krdona can you check whether you have the CI enabled for everyone with access in your fork? The pipeline is not registered somehow.

resolved all threads

added review-approved-point1 label

Is this MR ready to be merged? The pipeline status at the top still says "created" not "passed".

resolved all threads

mentioned in commit 383a317d

merged

mentioned in merge request !79297 (merged)