-
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
[TauHLT] Add Phase2 MuTau HLT #45976
[TauHLT] Add Phase2 MuTau HLT #45976
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45976/41735
|
A new Pull Request was created by @youngwan-kim for master. It involves the following packages:
@Martin-Grunewald, @cmsbuild, @mmusich can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This whole PR needs to be rebased on top of the changes at 0183839
...ation/python/HLT_75e33/paths/HLT_IsoMu20_eta2p1_LooseDeepTauPFTauHPS27_eta2p1_CrossL1_cfi.py
Outdated
Show resolved
Hide resolved
...ation/python/HLT_75e33/paths/HLT_IsoMu20_eta2p1_LooseDeepTauPFTauHPS27_eta2p1_CrossL1_cfi.py
Outdated
Show resolved
Hide resolved
...ation/python/HLT_75e33/paths/HLT_IsoMu20_eta2p1_LooseDeepTauPFTauHPS27_eta2p1_CrossL1_cfi.py
Outdated
Show resolved
Hide resolved
-hlt |
a62f79a
to
7ba44fd
Compare
This PR contains too many commits (368 >= 240) and will not be processed. |
7ba44fd
to
a62f79a
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45976/41743
|
Pull request #45976 was updated. @Martin-Grunewald, @cmsbuild, @mmusich can you please check and sign again. |
@cmsbuild, please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45976/41797 |
Pull request #45976 was updated. @Martin-Grunewald, @cmsbuild, @mmusich can you please check and sign again. |
@cmsbuild, please test |
+1 Size: This PR adds an extra 40KB to repository
Comparison SummarySummary:
|
for the record, due to this PR, the phase-2 HLT timing is up to 5632.6 ms from 5533.2 ms (or +1.8% relative increase) |
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. @rappoccio, @antoniovilela, @mandrenguyen, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
type tau |
+1 |
PR description:
This PR adds a Phase2 MuTau HLT menu with proper P2L1 seeds.
Relevant JIRA ticket : CMSHLT-3009
Relevant presentations :
PR validation: