-
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 support for L1T EtSum
s of type kZDC{P,M}
as trigger objects at HLT
#42707
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42707/36800
|
A new Pull Request was created by @missirol (Marino Missiroli) 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 |
urgent Because of #42708 (comment). |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b0de4f/34585/summary.html Comparison SummarySummary:
|
The DQM differences in the outputs of PR tests seem compatible with #39803, #39754 and #42525. I will relaunch the tests (now with a different IB as baseline) to verify that there are no DQM differences due to this PR. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b0de4f/34586/summary.html Comparison SummarySummary:
|
No DQM differences due to this PR in the latest round of tests, as expected. |
@@ -104,6 +107,7 @@ void HLTL1TSeed::fillDescriptions(edm::ConfigurationDescriptions& descriptions) | |||
desc.add<edm::InputTag>("L1JetInputTag", edm::InputTag("hltGtStage2Digis:Jet")); | |||
desc.add<edm::InputTag>("L1TauInputTag", edm::InputTag("hltGtStage2Digis:Tau")); | |||
desc.add<edm::InputTag>("L1EtSumInputTag", edm::InputTag("hltGtStage2Digis:EtSum")); | |||
desc.add<edm::InputTag>("L1EtSumZdcInputTag", edm::InputTag("hltGtStage2Digis:EtSumZDC")); |
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.
Let's hope L1T is using that label.
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.
For L1EtSumZdcInputTag
, I kept Zdc
as in the parameters of L1TGlobalProducer
(#42634). And yes, for :EtSumZDC
I plan to ask that they use this name when they update L1TRawToDigi
(unpacker).
+1 |
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 support for L1T
EtSum
s of typekZDC{P,M}
as trigger objects at HLT. These new types were introduced in #42634 (L1T-ZDC emulation).Two types,
trigger::TriggerL1ZDC{P,M}
, are added toTriggerTypeDefs.h
, and support for them is added to the pluginHLTL1TSeed
(which puts L1T objects into theedm::Event
as part of theTriggerFilterObjectWithRefs
data format).A similar update was done in #36951 to support the L1T data formats for "MuonShowers". One difference in this PR is that the underlying data format, i.e.
l1t::EtSum
, is not a new one (unlikel1t::MuonShower
was), and this makes the changes on the HLT side less extensive compared to #36951.Note: the
edm::InputTag
added toHLTL1TSeed
in this PR will point to a non-existing collection in the current HLT menus until an unpacker for the ZDC inputs to the L1T-uGT is added to CMSSW (see #42634 (comment)). For this reason, only aLogDebug
message is issued when this collection is missing, and eventually this should be replaced by aedm::LogWarning
(once the unpacker is available).No changes expected.
PR validation:
addOnTests.py
passed.If this PR is a backport, please specify the original PR and why you need to backport that PR. If this PR will be backported, please specify to which release cycle the backport is meant for:
CMSSW_13_2_X