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 GenHFHadronMatcher #25025
Fixed GenHFHadronMatcher #25025
Conversation
Avoid a segmentation fault in the case of a tau particle not having a mother particle. This was seen during nanoAOD processing.
Although the ParticleDataTable was pulled from the EventSetup, it was never used.
It was possible to change all member data of GenHFHadronMatcher to be const and the module was not calling any thread unsafe external functions. It was therefore easy to convert to a global module. In addition, it now used EDPutTokens and edm::Event::emplace which further improve the efficiency of the module.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_10_3_X. It involves the following packages: PhysicsTools/JetMCAlgos @cmsbuild, @fgolf, @peruzzim, @monttj can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
backport #25026 |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
please test workflow 10887.0,11087.0 |
The tests are being triggered in jenkins. |
-1 Tested at: 0914ef2 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
The relvals timed out after 2 hours. |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
please test workflow 10887.0,11087.0 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
Hi @Dr15Jones , for some reason, I can not find logs for 10887.0 and 11087.0 |
+1 |
merge |
The nanoAOD jobs were crashing in GenHFHadronMatcher because of a missing simulation parentage of a tau. This was fixed with the first commit.
Took the opportunity to modernize the module, especially since it is being used in an 8 thread job.