Handle menu chains outside of the config flags (HLT newJO)
This MR changes the way the menu chains are handled. They used to be part of the job flags which was suboptimal for several reasons. Now they are just put in a simple dict and passed between two functions.
Merge request reports
Activity
This merge request affects 6 files:
- Trigger/TriggerCommon/TriggerJobOpts/CMakeLists.txt
- Trigger/TriggerCommon/TriggerJobOpts/python/MenuConfigFlags.py
- Trigger/TriggerCommon/TriggerJobOpts/python/TriggerConfigFlags.py
- Trigger/TriggerCommon/TriggerJobOpts/share/runHLT_standalone_newJO.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/GenerateMenuMT_newJO.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1_newJO.py
Adding @sutt ,@hrussell ,@malconad ,@ckaldero ,@carquin ,@ademaria ,@bernius as watchers
added Trigger TriggerMenu master review-pending-level-1 labels
CI Result SUCCESS (hash b34edeb2)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 23258]added review-approved label and removed review-pending-level-1 label
added 184 commits
-
b34edeb2...3fc68229 - 183 commits from branch
atlas:master
- 97bc86b9 - Resolve conflict
-
b34edeb2...3fc68229 - 183 commits from branch
This merge request affects 6 files:
- Trigger/TriggerCommon/TriggerJobOpts/CMakeLists.txt
- Trigger/TriggerCommon/TriggerJobOpts/python/MenuConfigFlags.py
- Trigger/TriggerCommon/TriggerJobOpts/python/TriggerConfigFlags.py
- Trigger/TriggerCommon/TriggerJobOpts/share/runHLT_standalone_newJO.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/GenerateMenuMT_newJO.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1_newJO.py
Adding @sutt ,@hrussell ,@malconad ,@ckaldero ,@carquin ,@ademaria ,@bernius as watchers
added review-pending-level-1 label and removed review-approved label
CI Result SUCCESS (hash 97bc86b9)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 23334]added review-approved label and removed review-pending-level-1 label
mentioned in commit b3665d92
added sweep:ignore label