PerfMonComps: Enable flake8
This MR is a follow-up to !38852 (merged) and relates to ATEAM-678. We're enabling flake8
for the python modules and scripts under Control/PerformanceMonitoring/PerfMonComps
. No new functionality is introduced. There is a minor bug-fix in the way PerfMonSvc
obtains the list of algorithms. For example, in q431
the number of algorithms running were being reported to be zero:
Py:PerfMonSvc INFO job had [0] algorithms
now this is corrected:
Py:PerfMonSvc INFO job had [377] algorithms
Merge request reports
Activity
This merge request affects 1 package:
- Control/PerformanceMonitoring/PerfMonComps
This merge request affects 14 files:
- Control/PerformanceMonitoring/PerfMonComps/CMakeLists.txt
- Control/PerformanceMonitoring/PerfMonComps/python/DomainsRegistry.py
- Control/PerformanceMonitoring/PerfMonComps/python/JobOptCfg.py
- Control/PerformanceMonitoring/PerfMonComps/python/MTJobOptCfg.py
- Control/PerformanceMonitoring/PerfMonComps/python/PMonSD.py
- Control/PerformanceMonitoring/PerfMonComps/python/PerfMonFlags.py
- Control/PerformanceMonitoring/PerfMonComps/python/PerfMonSerializer.py
- Control/PerformanceMonitoring/PerfMonComps/python/PyComps.py
- Control/PerformanceMonitoring/PerfMonComps/python/PyMonUtils.py
- Control/PerformanceMonitoring/PerfMonComps/python/PyPerfMon.py
- Control/PerformanceMonitoring/PerfMonComps/share/FastMon.py
- Control/PerformanceMonitoring/PerfMonComps/share/FullMon.py
- Control/PerformanceMonitoring/PerfMonComps/share/PerfMonMTSvc_jobOptions.py
- Control/PerformanceMonitoring/PerfMonComps/share/PerfMonSvc_jobOptions.py
added Core master review-pending-level-1 labels
CI Result SUCCESS (hash a74b1112)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 25234]added review-approved label and removed review-pending-level-1 label
mentioned in commit d16f281f
added sweep:ignore label