-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Store DeepTauID in NanoAOD (backport for UL) #27227
Store DeepTauID in NanoAOD (backport for UL) #27227
Conversation
A new Pull Request was created by @mbluj for CMSSW_10_6_X. It involves the following packages: PhysicsTools/NanoAOD @cmsbuild, @santocch, @fgolf, @peruzzim 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. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+xpog as in #27226 (comment) |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_11_0_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 as agreed with XPOG and PPD |
PR description:
This PR is to store novel DeepTauIDs computed at MiniAOD step to NanoAOD tables.
It can be seen as an follow up action announced here: #27146 (comment)
Expected changes: six new variables are added to tau table in NanoAOD - three raw values (float numbers) of tau discriminants against jets, electrons and muons misidentified as taus, and three integer numbers encoding corresponding WPs.
NanoAOD for old eras should not be modified and should work on top of corresponding MiniAOD inputs not containing DeepTauID variables.
This PR is backport of #27226 to 10_6_X for UL.
PR validation:
NanoAOD workflows were tested.
if this PR is a backport please specify the original PR:
Backport of #27226.