Fix Tag and Probe monitors by using both electron branches
This MR fixes a bug identified in the Zee/JPsiee Tag and Probe Monitors where one leg was never used. It also implements a TH2D for electron monitoring to check ~% difference in eta electron monitoring between run2 and run3 egamma monitoring
Merge request reports
Activity
This merge request affects 1 package:
- Reconstruction/egamma/egammaPerformance
This merge request affects 3 files:
- Reconstruction/egamma/egammaPerformance/python/SetupEgammaMonitoring.py
- Reconstruction/egamma/egammaPerformance/src/MonitorElectronAlgorithm.cxx
- Reconstruction/egamma/egammaPerformance/src/MonitorTnPAlgorithm.cxx
Adding @goetz as watcher
added Egamma Reconstruction review-pending-level-1 labels
CI Result SUCCESS (hash 41355989)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 39345]added review-approved label and removed review-pending-level-1 label
Hi @pberta , it would be nice to have this in the reprocessing. However 22.0.41.5 has been built and is being validated now by PhysicsValidation and by DP/DQ for the reprocessing. We DP/DQ do not think we want a new release to be built just to include this gamma monitoring update. However if there is any discovery in the near future that require a new release to be built, we will be happy to have this egamma monitoring update to be included.
Thank you. Sarah, Song-Ming (tagging @demers , @perjohan , @sara , @smwang )
mentioned in commit e8460554
added sweep:done label
picked the changes into the branch
cherry-pick-e84605548f3-master
with commit 76c6fd85mentioned in commit 76c6fd85
Sweep summary
successful:- master
(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/1690/console for details)
mentioned in merge request !46655 (merged)
mentioned in commit a380e497