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
fix filling of pythia userhooks #34184
Conversation
please test |
A new Pull Request was created by @agrohsje for CMSSW_11_2_X. It involves the following packages: GeneratorInterface/Pythia8Interface @SiewYan, @alberto-sanchez, @agrohsje, @mkirsano, @GurpreetSinghChahal can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@smrenna backport to 11_2 FYI. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-34f80e/16107/summary.html Comparison SummarySummary:
|
+generators |
This pull request is fully signed and it will be integrated in one of the next CMSSW_11_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR is a back-port of this pull request from steve mrenna: #34068
we need a back-port to the current run 3 cmssw release which is still 11_2.
the original PR was to fix a segmentation fault of P8 when using userhooks. the code crashed at the start of a new lumisection.
I tested that the PR helps to avoid the crash and TSG-Run3Winter21GS-00071 runs fine. More tests already done for the master PR.