forked from cms-sw/cmssw
-
Notifications
You must be signed in to change notification settings - Fork 27
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
uGT emulator code for hadronic shower triggers for Run-3 #951
Merged
rekovic
merged 20 commits into
cms-l1t-offline:l1t-integration-CMSSW_12_0_2
from
dildick:l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
Nov 16, 2021
Merged
uGT emulator code for hadronic shower triggers for Run-3 #951
rekovic
merged 20 commits into
cms-l1t-offline:l1t-integration-CMSSW_12_0_2
from
dildick:l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
Nov 16, 2021
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
I ran these tests on lxplus
|
Lot of WFs did not run likely due to DAS errors |
@Nik-Menendez When you get to producing results for rate with the L1T menu framework, please add the results here. |
@rekovic Do you have any comments? |
…r' of github.com:dildick/cmssw into l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
…r' of https://github.com/dildick/cmssw into l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
@rekovic, when can we expect to have these changes merged? The menu team would need this to be in the ntuple recipe to proceed for the trigger review. Thank you! |
dildick
deleted the
l1t-integration-CMSSW_12_0_2-uGT-hadronic-shower-trigger
branch
November 17, 2021 17:35
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:
This PR is a first attempt to include the core code for the uGT emulator for hadronic shower triggers for Run-3. In previous PRs I introduced data formats for the CSC, EMTF and uGMT shower objects, and emulators for the CSC trigger, EMTF and uGMT. A new object MuonShower object is introduced in the L1TGlobal emulator and data format packages. This object is treated separately from the regular Muon object. The core code is currently switched off, i.e. the parameter that enables reading muon shower trigger bits from the L1T menu XML file is set to False. As such, there should be no differences in any workflow. The code is based on UTM lib 0.9.X (https://gitlab.cern.ch/cms-l1t-utm/utm), which was recently integrated into CMSSW_12_0_X (cms-sw/cmsdist#7337) and CMSSW_12_1_X (cms-sw/cmsdist#7281).
A few relevant presentations can be found here:
PR validation:
I tested the code on a private XML L1T menu, stripped from the regular trigger algorithms.
Additional validation with scram b runtests and with WF 11634.0.
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A
Before submitting your pull requests, make sure you followed this checklist: