Phase 2 HLT menu targetting 7.5×10³⁴ cm⁻²s⁻¹ #35342
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This configuration is based on - CMSSW 11.1.8 - global tag 111X_mcRun4_realistic_T15_v5 - the Tracking v6.1 baseline configuration - the TICL v3 configuration
The b-tagging paths should be consistent with the Jets/MET configuration.
Update the Jets/MET configuration: - replace PFPuppiMET with PFPuppiMETTypeOne - update the Jets/MET thresholds to match the AR and draft TDR - remove the PF CHS reconstruction
Remove unused sequences and modules: - BProbability for Puppi and PF jets, and their corresponding submodules; - HLTObjects and L1Objects; - DeepCSV and DeepJet paths for PFCHS jets, and their corresponding submodules. Merge all noFilter paths into MC_BTV. Various renames: - remove _v1 tags from path names; - rename L1T filters to match the implementation; - rename hltDoublePFPuppiJets128Eta2p3MaxDeta1p6_cfi to hltDoublePFPuppiJets128Eta2p4MaxDeta1p6_cfi because of wrong eta cut in name.
At the same time: - remove unused PSets and modules - clean up whitespcaes - clean up the imports in the main configuration file Co-authored-by: Thiago Tomei <thiago.tomei@cern.ch>
Add E/Gamma-related products to the input event content, and implement the L1 E/Gamma reconstruction. Implement the E/Gamma HLT reconstruction and trigger paths.
Update the L1T simulation: - update to the latest official recipe - enable L1T PF candidates - implement L1T NN taus
General configuration clean up: - remove duplicate or unused modules; - remove empty tasks; - remove unused or duplicate paths. Remove unused L1T simulation paths. Update the list of collections read form the input files.
Delete L1T and beam spot modules from the HLT configuration to avoid redefinition errors when running in the same process as the L1T simulation.
|
please test |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35342/25397
|
|
A new Pull Request was created by @fwyzard (Andrea Bocci) for master. It involves the following packages:
@jordan-martins, @missirol, @bbilin, @wajidalikhan, @Martin-Grunewald, @AdrianoDee, @srimanob, @kskovpen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
-1 Failed Tests: RelVals RelVals-INPUT RelValsExpand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ... |
|
Hmm, I see a maintenance issue in migrating forward - this time and in the future. The specific module now has this config: One could take such modules and their config from the combined table, but some (other) changes may be intentional for Phase-II! |
This was referenced Dec 7, 2021
This was referenced Jan 19, 2022
This was referenced Mar 2, 2022
This was referenced Mar 10, 2022
|
Any news or should this be closed for the moment? @fwyzard |
|
@trtomei is working on a new version of the HLT menu; if he agrees, I can close this PR and let him open a new one. |
|
@fwyzard I agree, please proceed. |
For reference, the new PR is #37162. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR description:
Implement the Phase-2 HLT menu used for the HLT TDR.
HLT_75e33_cfg.py implements a self-contained configuration:
HLT_75e33_cff.py implements a configuration fragment for use wih cmsDriver. To avoid redefinition errors, most of the L1T modules and the offline beamspot module are dropped from this configuration.
The HLT configuration is based on
CMSSW_11_1_8, forward-ported as-is to `CMSSW_12_1_0_pre3111X_mcRun4_realistic_T15_v5and implement the baseline Phase-2 configuration for Jets/MET, b-tagging, Muons and E/Gamma.
Taus are not included.
It includes "open" or "MC" HLT paths for running the reconstruction without any (pre)selection applied, and a partial re-emulation of the Level-1 Trigger, which simulates a simplified L1T menu using EDFilters and Paths.
Barring the changes in release and conditions, for Jets/MET and B-tagging the results obtained for the TDR correspond to the commit 38d7327 (and likely also with a6692af, which is just a technical fix), which still use "offline" muons for the Particle Flow reconstruction.
The following commits implement the use of HLT muons in the Particle Flow reconstruction, affecting the Jets/MET (and likely B-tagging) performance in a non-negligible way; for more details see this presentation.
For Muons, E/Gamma, and overall timing measurements, the results obtained in the TDR correspond to the full PR.
PR validation:
To be done.
if this PR is a backport please specify the original PR and why you need to backport that PR:
Forward port of #32903.