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 electron PFCandidates RefVector usage in MuonFSRProducer #28221
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28221/12348
|
A new Pull Request was created by @peruzzim for master. It involves the following packages: MuonAnalysis/MuonAssociators @cmsbuild, @santocch can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test workflow 1325.6,1325.7,1325.8,1329.1,136.7722,136.7952,136.8521 |
The tests are being triggered in jenkins. |
Thanks! |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@peruzzim did you explicitly verified the output of this fix? |
The tests here show no difference, in addition I've done a full set of tests in the nanoAOD suite (cms-nanoAOD#442), all tests successful and no change observed. |
+1 |
merge |
+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 be automatically merged. |
PR description:
This PR fixes the issue reported by @Dr15Jones in #28118 (review)
Should be backported to 102X and 106X.
PR validation:
Tested on one file, it runs. The code change is pretty minimal.