added a minimal series of muon jet chains for b-jet performance
Merge request reports
Activity
added BJetTrigger label
This merge request affects 1 package:
- Trigger/TriggerCommon/TriggerMenuMT
This merge request affects 3 files:
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/SignatureDicts.py
- Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py
Adding @sutt ,@dzanzi ,@iriu ,@carquin ,@ademaria ,@bernius ,@hrussell ,@malconad as watchers
added Trigger TriggerMenu master review-pending-level-1 labels
CI Result FAILURE (hash 577b3423) 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 37369] Hi @lfeligio we would prefer that additions to the menu files are done by menu experts. This is in particular true for additions of items to the L1 menu. For similar chains you have already added L1 items and we said we need to discuss them as we cannot add them all in the menu. So, please discuss with the bjet trigger coordinator and make a proposal for the full list of triggers you need and then we'll discuss on how to proceed.
Tagging @khoo
Hello @dzanzi, these additions are indeed the result of the discussion with b-jet trigger conveners. We understand, following the last exchange with you that L1 Topo mu-jet are not ready yet. In the meanwhile we think we should start looking at efficiencies and rate which in principle could be possible by starting from the proposed chains seeded by L1_MU4.
added 41 commits
-
577b3423...1a6faa74 - 38 commits from branch
atlas:master
- 9d6aa2da - added CTP ID for L1MU4_J15
- c7d27629 - Added CTP ID for L1_MU4_J15
- f3649611 - updated .ref file to account for new mu-jet chains
Toggle commit list-
577b3423...1a6faa74 - 38 commits from branch
Hi @lfeligio , after further private discussion, I understand for your studies it is sufficient to use MU4_J15 and emulate other possible triggers. So I would suggest to only have this in the L1 menu. Since I don't see other clients for the other MU+J chains, can you please remove these chains (including also those added in !45026 (closed)):
- 'L1_MU6_J20', 'L1_MU6_J40', 'L1_MU6_J75',
- 'L1_BTAG-MU4J20','L1_BTAG-MU4J30','L1_BTAG-MU6J20','L1_BTAG-MU6J25','L1_3J15_BTAG-MU4J15','L1_3J15_BTAG-MU4J30','L1_3J15_BTAG-MU6J25','L1_3J20_BTAG-MU4J20'
- revert the changes in !45026 (comment 4636070)
This leaves you with L1_MU4_J15 and L1_BTAG-MU4J15 (not working yet) to do the emulation.
Also, there is no need for new items to be included in the CTP ID map.
After these, you can flag this MR as ready, thanks
added 47 commits
-
fe1006b4...52bbf2f9 - 46 commits from branch
atlas:master
- bb3699c6 - added mu-jet HLT chains
-
fe1006b4...52bbf2f9 - 46 commits from branch
This merge request affects 3 packages:
- 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 changes-trigger-counts label
CI Result FAILURE (hash bb3699c6) 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 37505] added 1 commit
- a40543e2 - moved HLT mu-jet boffperf chains into combined slice