Fix ART for photons
The cleaning of IsolationSelection broke the ART tests for photons. This MR fixes this. It also improved slightly the egamma CI test to spot this kind of issues earlier
Merge request reports
Activity
added alsoTargeting:22.0 label
hi, just for the record, I think it'll probably be more practical (for MRs aiming both at 22.0 and master) to target the 22.0 branch (and then auto-sweep to master) rather than the contrary. But no need to change now since the pipeline started already, we'll see if the new alsoTargeting:22.0 label does its job properly.
Edited by Julien MaurerThis merge request affects 1 package:
- Reconstruction/egamma/egammaValidation
This merge request affects 4 files:
- Reconstruction/egamma/egammaValidation/share/egamma_art_checker_joboptions.py
- Reconstruction/egamma/egammaValidation/test/test_electron.sh
- Reconstruction/egamma/egammaValidation/test/test_electron_pileup.sh
- Reconstruction/egamma/egammaValidation/test/ut_egammaARTJob_test.sh
Adding @goetz as watcher
added Reconstruction review-pending-level-1 labels
CI Result SUCCESS (hash 78cb9cea)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
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 52965]added review-approved label and removed review-pending-level-1 label
mentioned in commit 78bf54b4
added sweep:done label
picked the changes into the branch
cherry-pick-78bf54b44f1-22.0
with commit 94fe8f27mentioned in commit 94fe8f27
Sweep summary
successful:- 22.0
(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/2664/console for details)
mentioned in merge request !53679 (merged)
mentioned in commit ee87f5cf