B-physics trigger: add HLT part for L1BPH-0M10C-3MU3V topo (ATR-19638)

Three muons fitted to the common vertex, invariant mass < 10GeV

• 'HLT_3mu4_b3mu_noos_L1BPH-0M10-3MU3V' : no charge check, invariant mass < 10GeV
• 'HLT_3mu4_b3mu_L1BPH-0M10C-3MU3V' : total charge = 1, invariant mass < 10GeV

added Trigger TriggerMenu changes-trigger-counts master labels

Thanks Vladimir

Can you add also the invariant mass cut at the HLT and remove the existing chains from the same L1 trigger that I think are now in the muon signature?

added 435 commits

• 14ef716a...9ff8baf5 - 431 commits from branch atlas:master
• f3ac3b17 - update MR
• 3e095a14 - Merge remote-tracking branch 'upstream/master' into master-b3mu
• 48edf3ac - bug fix
• c105c45f - update ref

Compare with previous version

added 1 commit

• b5647c54 - fix empty line

Compare with previous version

marked this merge request as ready

changed the description

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

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

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

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

This merge request affects 4 packages:

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

Affected files list will not be printed in this case

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

added review-pending-level-1 label

Hi @dzanzi,

I removed the original chains from the muon slice. Indeed, the existing 'b3mu' chains contain the invariant mass check, i.e. M < 10GeV.

Probably the name 'b3mu' is not self-explaining.

I am also thinking about 'noos' key which means accept all charge combination, or more explicitly do not select only opposite-sign combination. It also looks a little bit confusing, but I couldn't offer a more relevant name. We used this notation for run2 dimuon chains, so we can keep it for run3.

But if you have an idea how these keys should be named please let us know.

Vladimir.

all good thanks

CI Result FAILURE (hash b5647c54)

	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 1, 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 39857]

Jenkins please retry a build

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

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

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

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

This merge request affects 4 packages:

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

Affected files list will not be printed in this case

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

CI Result SUCCESS (hash b5647c54)

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

Looks good. Approved

Sergei L1

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

merged

mentioned in commit 119ba186

added sweep:ignore label