Enable new L1 menu format in L1Topo and CTP simulation (II)
This is a second attempt of !38137 (merged)
This was already reviewed and approved, but reverted (!38728 (merged)) due to the interference with the Trigger plans for a validation campaign.
cc @jpanduro , @okumura , @fwinkl, @rbielski , @smh , @wiedenma
PS: in addition this MR contains a commit that enables L1Topo simulation when trigger flag enableL1Phase1
is set True (addresses ATR-22459)
Merge request reports
Activity
This merge request affects 8 packages:
- HLT/Trigger/TrigMonitoring/TrigOnlineMonitor
- Trigger/TrigConfiguration/TrigConfData
- Trigger/TrigConfiguration/TrigConfigSvc
- Trigger/TrigT1/L1Topo/L1TopoCoreSim
- Trigger/TrigT1/L1Topo/L1TopoSimulation
- Trigger/TrigT1/TrigT1CTP
- Trigger/TriggerCommon/TriggerJobOpts
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @jchapman ,@sutt ,@kwhalen ,@wiedenma ,@asonay ,@orlando ,@cmorenom ,@mark ,@hrussell ,@fwinkl ,@ckaldero ,@vpascuzz ,@bernius ,@rbielski ,@afaulkne as watchers
added TriggerMenu label
CI Result SUCCESS (hash d6e2d0a7)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 24609]added review-pending-level-1 label
Dear reviewer. Just a reminder, the almost exact same code was already approved in !38137 (merged). The only addition was the last commit, which therefor should undergo a check.
Cheers, Joerg
added review-approved label and removed review-pending-level-1 label
mentioned in commit 3e754a82
added sweep:ignore label