Update e/y menu and monitoring configurations

This is a MR to update the menu (with few additional chains) and monitoring configurations for both legacy and MT aiming for the next round of validation. This includes Synchronized lists of monitored chains and chains in menu as in ATR-21430:

• Primary and lowest eT dielectron trigger (with L1_2EM15VHI seed (previously unavailable))
• Single photon trigger g120_loose_L1EM22VHI
• Updated configuration for athenaMT monitoring to include new electron and photon isolated chains discussed in ATR-20756
• Updated configuration also in legacy monitoring missing in 8th round validation (ATR-21986)

@cjmeyer, @safarzad and Fer

added Egamma TriggerMenu master + 1 deleted label

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

200K in file Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref

208K in file Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref

This merge request affects 4 packages:

• Trigger/TrigMonitoring/TrigEgammaMonitoring
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TriggerTest
• Trigger/TriggerCommon/TriggerMenuMT

Adding @okumura ,@bernius ,@jpanduro as watchers

added DQ Trigger review-pending-level-1 labels

CI Result FAILURE (hash 66b5c145)

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

Hi @fernando,

the CI is failing with

HistogramFiller2D   ERROR Mismatch of provided vector sizes 2,4 for res_et_vs_eta
HistogramFiller2D   ERROR Mismatch of provided vector sizes 2,4 for res_et_vs_et

Can you please have a look?

Thanks, Volker (L1)

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

Mmm.... seems to be the monitoring.... I' ll check now. thanks!

Fer

I can' t reproduce the error :-(

I see the problem is when monitoring resolution vs eta and resolution vs et, but I can't find how that error is triggered as I don' t get that when I test locally on my events.

I guess for now (and as that is not crucial) I'll try to deactivate generation of those plots for now so we have these new config files and new chains for next round of validation

Fer

Hi all,

This is fialing bc there is an ERROR printed by the monitoring framework because of vectors mismatch. But is not makeing any crash. Can we mask those errors?

Otherwise @jodafons can you check where in the code we are provoking this? Are we ploting 2D resolutions actually?

Cheers! @cjmeyer and Fer

added 506 commits

• 66b5c145...4bb61362 - 504 commits from branch atlas:master
• bdd7688b - disabled detailed histograms
• 8ed40068 - Merge tag 'nightly/master/2020-10-12T2101' into UpdateMonConfigs

Compare with previous version

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

208K in file Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref

216K in file Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref

This merge request affects 4 packages:

• Trigger/TrigMonitoring/TrigEgammaMonitoring
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TriggerTest
• Trigger/TriggerCommon/TriggerMenuMT

Adding @okumura ,@bernius ,@jpanduro as watchers

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

Hi all,

Fixed the error printout basically by deactivating some detailed histograms we don't need now until we fix it. So we can make it to the next validation round with improved and updated monitoring configuration

Cheers!

@cjmeyer and Fer

CI Result SUCCESS (hash 8ed40068)

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

YYYyeeessss!!!!!

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

Can we merge it? We need this in so to make it in time for the next validation round.

merged