-
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
Addition of SingleEleNonIso HLT path of pT threshold 115 GeV to Phase2 HLT menu #42819
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42819/36939
|
A new Pull Request was created by @swagata87 (Swagata Mukherjee) for master. It involves the following packages:
@cmsbuild, @missirol, @mmusich, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
type egamma |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-dc24ab/34814/summary.html Comparison SummarySummary:
|
@rovere and @SohamBhattacharya let us know if you have any further comments on this PR. |
Swagata informed us that she's checking the paths in 13_3 RelVals. I'd wait for her conclusion. |
Hi @mmusich , no further comments on this PR. |
OK, thanks.
would be good to open a gitHub issue with the details of the problem (@swagata87). |
GH issue is opened here: #42850 |
+hlt
|
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, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR adds a new HLT path (L1-seeded single electron non-isolated path) to the Phase2 regular menu and Phase2 timing menu. Performance of this new path (efficiency, rate etc) was presented in this talk at the HLT upgrade meeting of 12 Sept 2023. This is in-line with the plans of EGamma POG as outlined in this talk (slide 17) at the 6th HLT upgrade workshop in December 2022.
PR validation:
runTheMatrix.py -l 24834.911
ran successfully.cmsDriver.py Phase2 -s HLT:75e33 --processName=HLTX --conditions auto:phase2_realistic_T21 --geometry Extended2026D95 --era Phase2C17I13M9 --customise SLHCUpgradeSimulations/Configuration/aging.customise_aging_1000 --filein=/store/relval/CMSSW_13_3_0_pre2/RelValZpToEE_m6000_14TeV/GEN-SIM-DIGI-RAW/PU_131X_mcRun4_realistic_v6_2026D98PU200-v1/2580000/015b1c6a-25be-4a3c-bca3-ae047eb03fbb.root --inputCommands='keep *, drop *_hlt*_*_HLT, drop triggerTriggerFilterObjectWithRefs_l1t*_*_HLT' -n 10 --nThreads 1 --no_exec
and thencmsRun Phase2_HLT.py
ran fine, and this is part of the output:More detailed checks of efficiency of the new path were made in
CMSSW_13_1_X
by running on this dataset[*] as described in the talk.[*]
/ZprimeToEE_M-6000_TuneCP5_14TeV-pythia8/Phase2Spring23DIGIRECOMiniAODPU200_Trk1GeV_131X_mcRun4_realistic_v5-v1/GEN-SIM-DIGI-RAW-MINIAOD
Backport of this PR is not necessary.
Tagging HLT upgrade convenors (@rovere, @SohamBhattacharya), EGM-HLT convenors (@DebabrataBhowmik, @ravindkv), EGM convenors (@cms-sw/egamma-pog-l2).