-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Protect against motherless leptons #39270
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39270/31918
|
A new Pull Request was created by @mseidel42 (Markus Seidel) for master. It involves the following packages:
@SiewYan, @mkirsano, @Saptaparna, @cmsbuild, @alberto-sanchez, @menglu21, @GurpreetSinghChahal can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
type bugfix |
please test |
urgent |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-bf7447/27231/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+1
|
merge |
PR description:
Fixes #39250
Note that the crash only occurs for old particle guns before 93b170d (12_3_X)
PR validation:
Ran 9000 events from
/eos/cms/store/relval/CMSSW_12_0_1/RelValSingleMuPt10/MINIAODSIM/120X_mcRun3_2021_realistic_v7-v1/10000/03e1cf94-8b28-40ad-8fde-2597e215d1a6.root
which crashed before as the single leptons did not have any mothers for older particle gun samples.