Many features added to JetMacher + JetHistoMatchedFiller

I added a few features to the existing JetMatcherAlg as well as JetHistoMatchedFiller in the offline MT JetMonitoring. Many of these were suggested in this post.

This includes:

• Matching PFlow to EM jet (offline to offline)
• Monitor not only the pT difference between matched jets, but also energy and mass difference as well as the responses for each variable. Additionally, we monitor the pT and eta of the reference (=offline) jets to be able to create 2D histograms in coarse bins of eta or pT. Note that I duplicated the configuration of the specifications & histograms for online and offline jets, so that we can configure them both individually, rather than having the same setup for both.
• We are able to specify the calibration scale of the probed (=usually HLT) jets, so we can check different calibration scales. This could easily be extended/modified to have this feature also (or only) for the reference jets.
• Before, the JetMatcher had a slight bug which caused it to sometimes match a jet to another one that had already been matched previously. This happened in 22 of 500 test events. This is now fixed.
• Mini bonus: adding JVT relevant variables for chains that have a JVT selection (if relevant?)

Tagging @jbossios @valentem

added 1 commit

• 49220a69 - remove printouts, adding response plots for matched jets

Compare with previous version

changed the description

resolved all threads

marked this merge request as ready

Jenkins please retry a build

This merge request affects 2 packages:

• Reconstruction/Jet/JetMonitoring
• Trigger/TrigMonitoring/TrigJetMonitoring

Affected files list will not be printed in this case

Adding @goetz as watcher

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

CI Result SUCCESS (hash 49220a69)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis
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
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 28869]

The thread is resolved and the code changes look fine, approving. Pienpen(L1)

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

merged

mentioned in commit bfd676f0

added sweep:ignore label