testing changes in TrigHypoCommonTools: extend the module for considering multiple hypos ATR-21565 ATR-23880

Now the ComboHypoTool is capable to implement the AND of multiple hypos. I added two new chains photon+MET that I used to test changes I made. Please note, the monitoring tool behaves in a slightly different way when the chain has multiple comboHypo to combine. In case we have N hypos, the module will:

1. create N different histogram;
2. the histograms for the processed events will have different number of entries, because the module stops as soon as a hypo fails the condition(s).

changed title from WIP: testing changes in TrigHypoCommonTools: extend the module for considering multiple hypos to WIP: testing changes in TrigHypoCommonTools: extend the module for considering multiple hypos ATR-21565

added 1 commit

• 8c8384ae - now using ToolArray

Compare with previous version

added 3 commits

• 9aaef0d6 - added new photon+MET chain with multiple comboHypos
• 07d75c26 - changes necessary to handle multiple comboHypos/chain
• e57f21b2 - updates

Compare with previous version

Hi folks,

I successfully run the following test: athena.py -c "setMenu='LS2_v1';doWriteBS=False;doWriteRDOTrigger=True;selectChains=['HLT_g25_tight_icalotight_xe40_cell_xe50_tcpufit_18dphiAB_18dphiAC_80mTAC_L1EM22VHI']" --imf --perfmon --threads=1 --evtMax=100 --filesInput=/cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/TriggerTest/valid1.410000.PowhegPythiaEvtGen_P2012_ttbar_hdamp172p5_nonallhad.merge.RDO.e4993_s3214_r11315/RDO.17533168._000001.pool.root.1,/cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/TriggerTest/valid1.410000.PowhegPythiaEvtGen_P2012_ttbar_hdamp172p5_nonallhad.merge.RDO.e4993_s3214_r11315/RDO.17533168._000002.pool.root.1 TriggerJobOpts/runHLT_standalone.py >| athena_debug.log 2>&1 &

and the job completed w/o errors and the expert-monitoring.root file was built properly.

I'm now running the minimal art tests to check what happens in the other chains that use a comboHypo.

Cheers,

Giani

added 3 commits

• d4292695 - added another photon+MET chain
• 91ccb346 - improvements in the logic and the handling of the monitorining hists
• 5c089d74 - clean up the code

Compare with previous version

added 1 commit

• ab1fc829 - fixed chain name

Compare with previous version

added 1 commit

• ff45a27d - clean up

Compare with previous version

added 1 commit

• cbbb11e3 - new ref files

Compare with previous version

marked this merge request as ready

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

104K in file Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py

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

This merge request affects 4 packages:

• Trigger/TrigHypothesis/TrigHypoCommonTools
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TriggerTest
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @sutt ,@ademaria ,@iriu ,@vmartin ,@okumura ,@carquin ,@dzanzi ,@bernius ,@hrussell ,@malconad as watchers

added Trigger TriggerMenu changes-trigger-counts master review-pending-level-1 labels

changed the description

resolved all threads

changed title from testing changes in TrigHypoCommonTools: extend the module for considering multiple hypos ATR-21565 to testing changes in TrigHypoCommonTools: extend the module for considering multiple hypos ATR-21565 ATR-23880

CI Result SUCCESS (hash cbbb11e3)

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