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
[12_2_X backport] fix in bc->e filter #37931
Conversation
A new Pull Request was created by @swagata87 (Swagata Mukherjee) for CMSSW_12_2_X. It involves the following packages:
@SiewYan, @mkirsano, @Saptaparna, @cmsbuild, @GurpreetSinghChahal, @alberto-sanchez can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
backport of #37869 |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9541e5/24692/summary.html Comparison SummarySummary:
|
Hi generator experts, |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
@swagata87 CMSSW_12_2_4 is now ready: https://github.com/cms-sw/cmssw/releases/CMSSW_12_2_4 |
Many thanks Qiang and Andrea. |
PR description:
See details in PR description of #37869
This is an exact backport of 37869.
The backport is being made following a discussion in today's PPD meeting, during this talk:
https://indico.cern.ch/event/1158486/#30-retraining-of-pf-electron-i
The plan is that once this backport PR is merged and a new 12_2_X release is made, EGM will request QCD b/c->e samples again, in 12_2_X. With those samples, we will check the performance of DNN-based PF ID.