Fixing first set of flake8 warnings in JetValidation
This fixes a first set of flake8 warnings in JetValidation as reported in ATLJETMET-1300.
Another MR with the remaining issues will be prepared but requires some discussion with the experts.
Merge request reports
Activity
This merge request affects 1 package:
- Reconstruction/Jet/JetValidation
This merge request affects 5 files:
- Reconstruction/Jet/JetValidation/python/CompareJetTestFilesExec.py
- Reconstruction/Jet/JetValidation/python/JetTestCheck.py
- Reconstruction/Jet/JetValidation/python/JetTesters.py
- Reconstruction/Jet/JetValidation/python/PhysicsValidationHistos.py
- Reconstruction/Jet/JetValidation/python/RTTConfig.py
Adding @goetz as watcher
added JetEtmiss Reconstruction master review-pending-level-1 labels
added 905 commits
-
0b5a58fe...2957b31d - 904 commits from branch
atlas:master
- b5d765e5 - commits from master
-
0b5a58fe...2957b31d - 904 commits from branch
CI Result SUCCESS (hash 0b5a58fe)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 27333]This merge request affects 1 package:
- Reconstruction/Jet/JetValidation
This merge request affects 6 files:
- Reconstruction/Jet/JetValidation/CMakeLists.txt
- Reconstruction/Jet/JetValidation/python/CompareJetTestFilesExec.py
- Reconstruction/Jet/JetValidation/python/JetTestCheck.py
- Reconstruction/Jet/JetValidation/python/JetTesters.py
- Reconstruction/Jet/JetValidation/python/PhysicsValidationHistos.py
- Reconstruction/Jet/JetValidation/python/RTTConfig.py
Adding @goetz as watcher
CI Result SUCCESS (hash 9f677ec5)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 27388]added review-approved label and removed review-pending-level-1 label
mentioned in commit ef06e7b2
added sweep:ignore label