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
fixed bug in UL16 PU profile #29431
fixed bug in UL16 PU profile #29431
Conversation
A new Pull Request was created by @gkrintir (Georgios Konstantinos Krintiras) for CMSSW_10_6_X. It involves the following packages: SimGeneral/MixingModule @cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
@gkrintir @silviodonato |
please test |
The tests are being triggered in jenkins. |
urgent |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_11_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
backport of #29433 |
+1 |
PR description:
This fixes the pileup scenario for the 2016 ultra legacy (periods B-H), i.e.,
"2016_25ns_UltraLegacy_PoissonOOTPU"
from #28595 noted by @srimanob @ameyer-cern and reported by TSG et al: @missirol @pgunnell @Sam-Harper @gennai
In the validation plot below:
black the pileup distribution from this PR
red the pileup 'all 2016' distribution from this presentation that can be found here too
As for the bug, I had checked that the file didn't break the workflow:
/afs/cern.ch/user/g/gkrintir/public/ForPileup
but indeed it seems I didn't carefully investigate
/afs/cern.ch/user/g/gkrintir/public/ForPileup/histProbFunction.root
Regards and very sorry for that once more :(