TrigSignatureMoniMT: refactoring

• add more documentation
• change names to be more meaningful
• move the histogram initialization parts to be more organized and readable
• reduce the usage of auto when unnecessary
• whitespaces

Tagging @tbold

This merge request affects 1 package:

• Trigger/TrigMonitoring/TrigSteerMonitor

This merge request affects 2 files:

• Trigger/TrigMonitoring/TrigSteerMonitor/src/TrigSignatureMoniMT.cxx
• Trigger/TrigMonitoring/TrigSteerMonitor/src/TrigSignatureMoniMT.h

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

CI Result SUCCESS (hash e070aedb)

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

You may also consider dropping the MT and renaming it to TrigSignatureMoni. But that can wait of course.

resolved all threads

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

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

mentioned in commit aa28bb83

merged

added sweep:ignore label

mentioned in merge request !41534 (merged)