Fix efficiency plots in TrigEgammaMonitoringMT

This MR is to fix some bugs and show the efficiency plots in the TrigEgammaMonitoringMT package. This implementation has been discussed in the JIRA ticket: ATR-22557 .

ping: @fernando , @cjmeyer ,@jodafons ,@safarzad and @mverissi .

added DQ Egamma Trigger master labels

This merge request affects 1 package:

• Trigger/TrigMonitoring/TrigEgammaMonitoring

This merge request affects 5 files:

• Trigger/TrigMonitoring/TrigEgammaMonitoring/python/TrigEgammaMonitoringMTConfig.py
• Trigger/TrigMonitoring/TrigEgammaMonitoring/src/TrigEgammaMonitorBaseAlgorithm.cxx
• Trigger/TrigMonitoring/TrigEgammaMonitoring/src/TrigEgammaMonitorBaseAlgorithm.h
• Trigger/TrigMonitoring/TrigEgammaMonitoring/src/TrigEgammaMonitorElectronAlgorithm.cxx
• Trigger/TrigMonitoring/TrigEgammaMonitoring/src/TrigEgammaMonitorTagAndProbeAlgorithm.cxx

Adding @jmeyer ,@dbakshig ,@fernando as watchers

added review-pending-level-1 label

CI Result SUCCESS (hash c6a7a9c4)

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

Great, it is pass all :) Thanks a lot @eegidiop , let's waiting for the revision. Just to mention: This MR is very important to the e/g offline monitoring.

Cheers, João

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

Thanks for this. But am a little confused: This seems only to change the name of the pidname working points. Do this make the tool fill the efficiency histograms only with that change?

yes kkk... A stupid bug...

merged

mentioned in commit 47dce2c6

added sweep:ignore label