Sort HLT and L1 histogram x-axes alphabetically
This addresses issues discussed in [ATR-21172] and [ATR-19976] by alphabetically sorting the x-axes of the histograms monitoring L1 items and HLT chains.
Currently, the histograms monitoring L1 items and HLT chains which are fired in the run, are filled in the order the events happen to be processed. This means that the shape of the histograms can differ quite dramatically for reasons purely to do with process ordering, which is clearly not the desired behaviour.
After the change, we will have these improvements:
- Reprocessing the same run without changing the menu will result in an identical histogram.
- If the menu is changed (by removing or adding a few chains) the x-axis will shift a little bit in a way which is clearly recognisable.
- All histograms from a particular signature will end up together. (With the exception of multi-ele/jet/muon/etc where HLT_2jX_blabla will be closer to HLT_2eX_blabla than to HLT_jX_blabla )
Example of desired behaviour (Run 2 code): https://atlas.web.cern.ch/Atlas/project/trigger/trigcaf/www/ATR-24194-legacy/ResultMon/L1Events.html
What is currently looks like in Run 3 code: https://atlas.web.cern.ch/Atlas/project/trigger/trigcaf/www/ATR-24194-run3/ResultMon/L1Events.html (data and reference disagree artificially even though there was no change).
Merge request reports
Activity
This merge request affects 1 package:
- Trigger/TrigMonitoring/TrigHLTMonitoring
This merge request affects 2 files:
- Trigger/TrigMonitoring/TrigHLTMonitoring/python/TrigGeneralMonitorAlgorithm.py
- Trigger/TrigMonitoring/TrigHLTMonitoring/src/TrigHLTMonitorAlgorithm.cxx
Adding @ebergeas as watcher
added review-pending-level-1 label
- Resolved by Volker Andreas Austrup
CI Result FAILURE (hash 923e27a9)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 39793]This merge request affects 1 package:
- Trigger/TrigMonitoring/TrigHLTMonitoring
This merge request affects 2 files:
- Trigger/TrigMonitoring/TrigHLTMonitoring/python/TrigGeneralMonitorAlgorithm.py
- Trigger/TrigMonitoring/TrigHLTMonitoring/src/TrigHLTMonitorAlgorithm.cxx
Adding @ebergeas as watcher
CI Result SUCCESS (hash 8f649208)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 39803]added review-approved label and removed review-pending-level-1 label
mentioned in commit 9ff8baf5
added sweep:ignore label