-
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
Add plugin to use existing HLTDoubleSinglet module with two PFJets and one RecoEcalCandidate #40994
Add plugin to use existing HLTDoubleSinglet module with two PFJets and one RecoEcalCandidate #40994
Conversation
…d one RecoEcalCandidate
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40994/34506
|
A new Pull Request was created by @brallmond (Braden Allmond) for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
@brallmond , the title of this PR is truncated, please fix it. Then, please update the title of the backport PR accordingly, e.g. title of backport = "(title of this PR) [ |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-96fa16/31141/summary.html Comparison SummarySummary:
|
+hlt Adds an HLT filter currently unused in any central wf. |
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 |
PR description:
To support future VBF Electron and VBF Photon HLTs in the scenario that VBF Parking is accepted (see JIRA https://its.cern.ch/jira/browse/CMSLITDPG-1099), one plugin is added to the existing plugins file of HLTrigger/HLTfilters.
PR validation:
With the changed file in place, the rates of VBF Electron and VBF Photon were estimated after changing by-hand the EDFilter type to use for the final filter of each path. It was found that events pass both paths and use the triplet filter in the expected way (cross-cleaning a RecoEcalCandidate from two PFJets).
NB: It wasn't obvious to me this was the case at first, but using the same EDFilter module for both electrons and photons is in-line with existing HLT paths and code (see the Ele-Tau cross trigger, which uses a HLT2PhotonTau EDFilter in its cross-cleaning module, hltOverlapFilterIsoEle24IsoTau30WPTightGsfCaloJet5).