-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Low pT electrons (up to GsfTracks) for 10_2_X #25680
Conversation
moved low pT ele sequence into highlevelreco
A new Pull Request was created by @bainbrid for CMSSW_10_2_X. It involves the following packages: Configuration/Eras @perrotta, @ssekmen, @lveldere, @civanch, @mdhildreth, @cmsbuild, @franzoni, @slava77, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
This PR depends on the BDT models included in cms-data/RecoEgamma-ElectronIdentification#10 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1
|
code-checks |
The code-checks are being triggered in jenkins. |
+1 |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25680/8221 |
+operations the update to have the similar functionality as in #25773 appears correct |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_5_X is complete. 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 the code, under validation in CMSSW_10_5_X (in a different tuning) is not activated in the normal Eras |
This PR is equivalent to the open #25455 PR, but rebased to 10_2_X using a very recent IB.
This PR was opened in response to this instruction.