Addition of PRW to IDPVM
This MR introduces the possibility to run pileup reweighting within IDPVM. This will allow immediate comparison between data and mc samples for tracking quantities.
The implementation works with mc23 samples for which the setup of PRW is simplified thanks to the introduction of common PRW files. The attached plot shows the impact on the actual mu distribution when running over 1000 ttbar mc23a events (black no prw, red prw).
Credits to @tstreble for the help with the implementation, tagging @jburzyns @skadir
Merge request reports
Activity
- Resolved by Guglielmo Frattari
- Resolved by Thomas Strebler
- Resolved by Guglielmo Frattari
- Resolved by Thomas Strebler
added review-user-action-required label
This merge request affects 1 package:
- InnerDetector/InDetValidation/InDetPhysValMonitoring
This merge request affects 5 files:
- InnerDetector/InDetValidation/InDetPhysValMonitoring/InDetPhysValMonitoring/InDetPhysValMonitoringTool.h
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValFlags.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValMonitoringConfig.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/scripts/runIDPVM.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/src/InDetPhysValMonitoringTool.cxx
Adding @goetz ,@bkerridg ,@keli ,@lgagnon ,@sroe as watchers
added InnerDetector main review-pending-level-1 labels and removed review-user-action-required label
added review-user-action-required label and removed review-pending-level-1 label
CI Result SUCCESS (hash 03b4ecb0)Athena externals cmake make 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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3092]added 2 commits
This merge request affects 1 package:
- InnerDetector/InDetValidation/InDetPhysValMonitoring
This merge request affects 5 files:
- InnerDetector/InDetValidation/InDetPhysValMonitoring/InDetPhysValMonitoring/InDetPhysValMonitoringTool.h
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValFlags.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValMonitoringConfig.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/scripts/runIDPVM.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/src/InDetPhysValMonitoringTool.cxx
Adding @lgagnon ,@bkerridg ,@keli ,@sroe ,@goetz as watchers
added review-pending-level-1 label and removed review-user-action-required label
This merge request affects 1 package:
- InnerDetector/InDetValidation/InDetPhysValMonitoring
This merge request affects 5 files:
- InnerDetector/InDetValidation/InDetPhysValMonitoring/InDetPhysValMonitoring/InDetPhysValMonitoringTool.h
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValFlags.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/python/InDetPhysValMonitoringConfig.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/scripts/runIDPVM.py
- InnerDetector/InDetValidation/InDetPhysValMonitoring/src/InDetPhysValMonitoringTool.cxx
Adding @bkerridg ,@goetz ,@keli ,@lgagnon ,@sroe as watchers
added Analysis CPAlgorithms analysis-review-required labels
CI Result SUCCESS (hash da95aa33)Athena externals cmake make 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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3110]added 1 commit
- 344e72a1 - moving prwtool with ca setup to another config