Legacy jet monitoring tool default chain plotting changed for AthenaMT samples

As discussed with @jbossios in this ticket: https://its.cern.ch/jira/browse/ATR-21819

I changed the legacy jet monitoring tool such that when running over AthenaMT input samples, the fillBasicHLTforChain function fills the default plots for "all jets" of the underlying jet container, while putting the "only jets that pass the chain criteria" into a subfolder chain_name+"/PassingJets"

Also tagging @khoo

unmarked as a Work In Progress

WARNING: big files (>100K) are found in the changeset

128K in file Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.cxx

This merge request affects 2 files:

• Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.cxx
• Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.h

added DQ JetEtmiss Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash 76273f7f)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22685]

Changes mostly look fine, just added one comment above. Tests pass, so once the above thread is resolved we can approve this. -Mike (L1)

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 1309cb81 - removing commented out lines

Compare with previous version

resolved all threads

WARNING: big files (>100K) are found in the changeset

128K in file Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.cxx

This merge request affects 2 files:

• Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.cxx
• Trigger/TrigMonitoring/TrigJetMonitoring/src/HLTJetMonTool.h

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash 1309cb81)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 22703]

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

merged

mentioned in commit ab4e037c

added sweep:ignore label