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
Avoid printing statistics if hadronizer has not been initialized properly #25579
Avoid printing statistics if hadronizer has not been initialized properly #25579
Conversation
…erly Also add a unit test for Pythia8 to confirm this change prevents a crash.
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25579/7818 |
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages: GeneratorInterface/Core @alberto-sanchez, @cmsbuild, @qliphy, @perrozzi, @efeyazgan can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
ping |
code-checks |
The code-checks are being triggered in jenkins. |
@qliphy @efeyazgan @alberto-sanchez the added test is successful, and the code update looks reasonable, could you please review and sign or comment? |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25579/8002
|
@qliphy @efeyazgan @alberto-sanchez the added test is successful, and the code update looks reasonable, could you please review and sign or comment? |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
AddOn tests in #25577 showed a segfault in
Pythia8::Pythia::stat()
when an exception was thrown by another module inbeginRun()
A unit test with
TestProcessor
(included) confirms the segfault in case theRun
has noLuminosityBlock
s (in which case the Pythia8's initialization has not been called). This PR suggests to add a boolean flag toGeneratorFilter
template to avoid calling theendRun
statistics printouts if the hadronizer+decayer have not been initialized (assuming theendRun
printout is really needed).Tested in 10_4_0_pre4, no changes expected.
@Dr15Jones