ATR-22117 add random trigger for FIRSTEMPTY
- add L1_RD0_FIRSTEMPTY and HLT_noalg_cosmiccalo_L1RD0_FIRSTEMPTY item to menu
- add CTP ID info print functionality
Merge request reports
Activity
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 ,@okumura ,@carquin ,@bernius ,@hrussell ,@ckaldero ,@malconad ,@jpanduro as watchers
added Trigger TriggerMenu changes-trigger-counts master review-pending-level-1 labels
CI Result SUCCESS (hash 72f4418c)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 1
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 25022]The warnings are because you are using
print
rather thanlogging
/var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py:14:5: warning: ATL901: use 'AthenaCommon.Logging' instead of 'print' print("There are %d available CTP IDs:"%len(available),",".join(map(str,available))) ^ /var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py:15:5: warning: ATL901: use 'AthenaCommon.Logging' instead of 'print' print("IDs >= 472 go in partition 2, IDs >= 492 go in partition 3\n") ^ /var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py:16:5: warning: ATL901: use 'AthenaCommon.Logging' instead of 'print' print("There are %d free items"%freeItems,"\n") ^ /var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py:17:5: warning: ATL901: use 'AthenaCommon.Logging' instead of 'print' print("There are %d floating items:"%len(floatingItems),",".join(map(str,floatingItems)),"\n") ^ /var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Menu/Menu_MC_pp_v8.py:18:5: warning: ATL901: use 'AthenaCommon.Logging' instead of 'print' print("There are %d unused items with CTP ID:"%len(unusedItemsWithCTPID),",".join(map(str,unusedItemsWithCTPID))) ^ warning: Python code violations found. See <https://gitlab.cern.ch/atlas/atlasexternals/tree/master/External/flake8_atlas>.
Please can you switch?
Tom - L1 shifter
added review-user-action-required label and removed review-pending-level-1 label
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 ,@okumura ,@carquin ,@bernius ,@hrussell ,@ckaldero ,@malconad ,@jpanduro as watchers
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash 3f766987)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 25055]- Resolved by Stefanie Morgenstern
added review-user-action-required label and removed review-pending-level-1 label
added review-pending-level-1 label
removed review-user-action-required label
added review-approved label and removed review-pending-level-1 label
mentioned in commit 220052b9