-
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
Code to process phase2 version of deepTauID v2p5 #40622
Code to process phase2 version of deepTauID v2p5 #40622
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40622/33897 |
A new Pull Request was created by @mbluj for master. It involves the following packages:
@cmsbuild, @mandrenguyen, @clacaputo, @swertz, @vlimant can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hello, |
please test with cms-data/RecoTauTag-TrainingFiles#11 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6b4ecf/30241/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
All automated tests seem be OK. Is there anything we can do to help with review of this PR? |
+xpog Expected changes in tauID products for Phase2 MINIAODs. Thanks for the reminder @mbluj |
+reconstruction |
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 now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Just a reminder: this PR requires that also data-model PR cms-data/RecoTauTag-TrainingFiles#11 is merged and then propagated to cms-dist. |
PR description:
This PR adds code for a phase-2 version of DeepTau discriminator. The network structure is identical to that of Run-2/3 DeepTau v2p5, therefore code modifications are small. The changes include:
slimmedElectrons
(gsfElectrons in EB) andslimmedElectronsHGC
(HGCal electrons in endcaps). Notes: 1. The latter collection is calledslimmedElectronsFromMultiCl
in older CMSSW (<13_0) release series (and samples produced with them) which can cause some problems with running on old samples; 2. The same merged electron collection is used by old-style anti-electron tau discriminant for phase-2;The performance of this phase 2 DeepTau discriminator is documented in AN-22-090.
The PR to cms-data with the data-model is here: cms-data/RecoTauTag-TrainingFiles#11
Backport of the PRs to 12_5/12_6 is foreseen.
To run standalone test, one can run the configuration file
RecoTauTag/RecoTau/test/runDeepTauIDsOnMiniAOD.py
after setting the flagphase2 = True
in that file.PR validation:
Validated with the standalone test mentioned above and with miniAOD workflows, matrix tests,
runTheMatrix.py -l limited -i all --ibeos
, successful.Note: the tests should be run with data-model here: cms-data/RecoTauTag-TrainingFiles#11