Improve HLT jet monitoring, and use JetRecAlg more widely

Various improvements:

• Renamed some monitored variables for clarity
• Tweaked histogram ranges
• Replaces outdated JetAlgorithm+JetRecTool with new JetRecAlg (some additional modifiers were needed, and the odd config change)

Attn @jbossios, @valentem -- do we want to discuss in more detail what we want to have monitored? Among other things I note that we don't have any monitoring of the jet mass, which might be relevant for some chains. There was also discussion of pileup-dependent histograms.

added 1 commit

• b9abfc1f - Add monTool for copy collections as well

Compare with previous version

marked this merge request as ready

added 2 commits

• 57100cf2 - Syncing up monitored variable names, add jet mass
• e9358502 - Improve RC jet naming, grooming to JetRecAlg

Compare with previous version

This merge request affects 5 packages:

• Reconstruction/Jet/JetRec
• Reconstruction/Jet/JetRecConfig
• Reconstruction/Jet/JetRecTools
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@mleblanc ,@bernius as watchers

added JetEtmiss Reconstruction Trigger TriggerEDM TriggerMenu master review-pending-level-1 labels

This merge request affects 5 packages:

• Reconstruction/Jet/JetRec
• Reconstruction/Jet/JetRecConfig
• Reconstruction/Jet/JetRecTools
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@mleblanc ,@bernius as watchers

CI Result FAILURE (hash e9358502)

	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 1
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 27109]

CI Result FAILURE (hash b9abfc1f)

	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 27107]

resolved all threads

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

resolved all threads

added 774 commits

• e9358502...4882b1c2 - 768 commits from branch atlas:master
• 44e74374 - Remove unused python var
• bdffcee9 - Update HLT RC jet name
• 89d86fb6 - Merge branch 'master' of https://gitlab.cern.ch:8443/atlas/athena into master-HLT-JetMonTiming
• 51672c27 - Update constit mod to apply EM cluster energy
• cdab1984 - Merge branch 'master' of https://gitlab.cern.ch:8443/atlas/athena into master-HLT-JetMonTiming
• fabfac65 - Update random seeding option for JetClusterer

Compare with previous version

This merge request affects 6 packages:

• Reconstruction/Jet/JetRec
• Reconstruction/Jet/JetRecConfig
• Reconstruction/Jet/JetRecTools
• Trigger/TrigMonitoring/TrigJetMonitoring
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@mleblanc ,@bernius as watchers

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

CI Result FAILURE (hash fabfac65)

	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 27846]

added 1 commit

• 26af1ec7 - Add Jvt to decorations propagated to presel view container

Compare with previous version

This merge request affects 6 packages:

• Reconstruction/Jet/JetRec
• Reconstruction/Jet/JetRecConfig
• Reconstruction/Jet/JetRecTools
• Trigger/TrigMonitoring/TrigJetMonitoring
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @goetz ,@sutt ,@hrussell ,@okuprash ,@dzanzi ,@mleblanc ,@bernius as watchers

CI Result SUCCESS (hash 26af1ec7)

	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 27873]

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

merged

mentioned in commit ce3cb4f9

added sweep:ignore label