New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
puppi bug fix for low PU case #9338
puppi bug fix for low PU case #9338
Conversation
A new Pull Request was created by @nhanvtran for CMSSW_7_5_X. puppi bug fix for low PU case It involves the following packages: CommonTools/PileupAlgos @cmsbuild, @cvuosalo, @vadler, @monttj, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
@nhanvtran: Is there any simple way to verify if your bug fix is working? What test procedure would show its effectiveness? The Jenkins tests show no differences. |
+1 A PUPPI bug fix to cover the case of very low pile-up. The code changes are satisfactory, and Jenkins tests showed no significant differences before they were mysteriously deleted. Tests of the Mini-AOD step of workflow 25202.0_TTbar_13 against baseline CMSSW_7_5_X_2015-05-28-2300 show little change in product size and some improvements in CPU time:
|
puppi bug fix for low PU case
Hi, a small bug fix for PUPPI in the case of very low PU (< 5 nPU) that was not included in the first PR for 75X.
Thanks