PerformanceMonitoring: First batch of cmake, flake8, clean-up updates
This relates to ATEAM-678. The original goal was to enable flake8
for the packages under Control/PerformanceMonitoring
. While doing so, I tried to clean things up that I'm fairly certain not used and/or needed anymore, although I played it safe. I also went through the cmake
configurations, fixed a number of py3
problems in some scripts that are not used very frequently etc. It turned out to be a bit more massive MR than I was intending but pretty much all changes are either stylistic and/or uncontroversial. There is essentially no new functionality.
BTW, there are a few commits but it would be nice if we don't squash them just in case.
Merge request reports
Activity
This merge request affects 4 packages:
- Control/PerformanceMonitoring/PerfMonAna
- Control/PerformanceMonitoring/PerfMonEvent
- Control/PerformanceMonitoring/PerfMonGPerfTools
- Control/PerformanceMonitoring/PerfMonVTune
Affected files list will not be printed in this case
added Core master review-pending-level-1 labels
This merge request affects 4 packages:
- Control/PerformanceMonitoring/PerfMonAna
- Control/PerformanceMonitoring/PerfMonEvent
- Control/PerformanceMonitoring/PerfMonGPerfTools
- Control/PerformanceMonitoring/PerfMonVTune
Affected files list will not be printed in this case
CI Result SUCCESS (hash 2a1a6d6b)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 24610]- Resolved by Alaettin Serhan Mete
CI Result SUCCESS (hash cde11f98)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 24612]- Resolved by Liza Mijovic
added review-pending-level-2 label and removed review-pending-level-1 label
added review-approved label
removed review-pending-level-2 label
mentioned in merge request !39028 (merged)
mentioned in commit 95df2e0b
added sweep:ignore label
mentioned in merge request !39165 (merged)