B-physics migration of combined dielectron+muon chains, e.g. 'HLT_e9_lhvloose_e5_lhvloose_bBeeM6000_mu4_L1BPH-0M9-EM7-EM5_MU6' (ATR-23839)

• TrigMultiTrkComboHypo is adopted to look for close-by electrons originated from the same EM cluster (chains with 'L1BPH-0DR3-EM7J15')
• combined dielectron+muon are added to LS2_v1 menu, but we are still waiting for ATR-23965; hack of the auto-generated IMEmptyMuonSeq1_Electron can be used in the meantime (add ~lyubushk/public/Trigger/fixIMEmptySeq_bBeeM6000.py to --postInclude option, see ~lyubushk/public/Trigger/test_bBeeM6000.sh for example);
• add more options for Bmumux decays of B_c+ and fix some bugs;
• add entry for di-electron chains in EDM;
• some improvements in the coding style are taken from !44579 (closed)

CC: @abarton

added Trigger TriggerMenu changes-trigger-counts master labels

added 1 commit

• 5c30f885 - remove fake decisions from TrigMultiTrkComboHypo

Compare with previous version

added 1575 commits

• 5c30f885...eb24eb1f - 1572 commits from branch atlas:master
• 529b5d37 - fix transitional errors, r2021-08-19T2101
• 1192b392 - temporary commit
• 2e19ae38 - Merge remote-tracking branch 'upstream/master' into master-bBee

Compare with previous version

added 1 commit

• c1aaa237 - fix B_c+ to J/psi D+(to K- pi+ pi+) like chains

Compare with previous version

added 1 commit

• 1fa94124 - updates from master-tau3mu

Compare with previous version

added 1 commit

• 648f810b - updates from tau3mu

Compare with previous version

added 144 commits

• 648f810b...4fd5786d - 143 commits from branch atlas:master
• 5f04069e - merge with upstream

Compare with previous version

added 1 commit

• c293755b - add EDM for di-electron chains

Compare with previous version

added 1 commit

• e5cc1e6c - update ref files

Compare with previous version

marked this merge request as ready

changed the description

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

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

100K in file Trigger/TrigValidation/TrigP1Test/share/ref_v1Dev_decodeBS_build.ref

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

This merge request affects 5 packages:

• Trigger/TrigHypothesis/TrigBphysHypo
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TrigP1Test
• Trigger/TriggerCommon/TrigEDMConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @ademaria ,@dbakshig ,@vmartin ,@okumura ,@carquin ,@abarton ,@bernius ,@okuprash ,@sutt ,@dzanzi ,@lyubushk ,@salderwe ,@iriu ,@fernando ,@hrussell ,@jodafons ,@malconad ,@cjmeyer as watchers

added Egamma TriggerEDM review-pending-level-1 labels

CI Result SUCCESS (hash e5cc1e6c)

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

Hi @lyubushk,

only one small comment from L1 side. Given the size of the MR, I think it would be good, if L2 could also take a quick look, in case I missed anything.

Cheers, Volker (L1)

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

resolved all threads

resolved all threads

MR looks good. Approving.

Cheers, Abhishek [L2]

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

merged

mentioned in commit 7c95b060

added sweep:ignore label