-
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
FirstVersionsofDiTauHLTPaths_forPhase2 [13_2_X] #42652
FirstVersionsofDiTauHLTPaths_forPhase2 [13_2_X] #42652
Conversation
…nd DeepTau Isolation (using Run2 training) are added.
A new Pull Request was created by @hsert (HaleSert) for CMSSW_13_2_X. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@hsert , the title of this PR is incomplete ("..."). Maybe take this opportunity to change the titles of your 3 PRs as follows: "foo" (master), "foo [13_2_X]" (this PR), and "foo [13_1_X]", where "foo" is the title of the PR to master. |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-acbd1c/34461/summary.html Comparison SummarySummary:
|
+hlt |
This pull request is fully signed and it will be integrated in one of the next CMSSW_13_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_3_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
First versions of ditau HLT paths for Phase2 with charged isolation and DeepTau Isolation (using Run2 training) are added.
Detailed description of PR is given in the original PR: #42562
We would like that PR will be backported to CMSSW_13_2_X release, to be compatible wtih other Phase 2 HLT PRs.
I used CMSSW_13_2_X_2023-08-18-1100 release for testing. After initialization, the merged branch is rebased with
git cms-rebase-topic --old-base CMSSW_13_3_X_2023-08-15-1100 hsert:tauHLTPhase2_dev
and run the following runMatrix test, and there was no failed run.
runTheMatrix.py -l limited -i all --ibeos