TriggerMenuMT: remove usage of TriggerFlags for menu generation
Remove the usage of TriggerFlags.XYZSlice.signatures
in the menu
generation. The only hook to modify the list of signatures externally is
the setChainFilter
method. So there is no need to even expose the list
of signatures outside the TriggerMenuMT
package via flags:
- Introduce
ChainStore
class/dict that holds the list of chains per signature. Also takes care of only allowing a fixed set of signature names. -
setupMenu()
return aChainStore
that is used inGenerateMenuMT
. - Adapt
runHLT_standalone.py
to the chain filter changes.
Closes ATR-24201.
Merge request reports
Activity
added Trigger TriggerMenu master review-pending-level-1 labels
- Resolved by Frank Winklmeier
- Resolved by Daniele Zanzi
CI Result FAILURE (hash a41f2a93)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 39591]added review-user-action-required label and removed review-pending-level-1 label
added 57 commits
-
a41f2a93...598aebb9 - 56 commits from branch
atlas:master
- 404fc64a - TriggerMenuMT: remove usage of TriggerFlags for signature generation
-
a41f2a93...598aebb9 - 56 commits from branch
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash 404fc64a)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 39655]- Resolved by Frank Winklmeier
Hi,
this caught merge conflicts, unfortunately. Setting review-user-action-required.
Cheers, Volker (L1)
added review-user-action-required label and removed review-pending-level-1 label
added 24 commits
-
404fc64a...a8fa2f71 - 23 commits from branch
atlas:master
- cb3f8f01 - TriggerMenuMT: remove usage of TriggerFlags for signature generation
-
404fc64a...a8fa2f71 - 23 commits from branch