-
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
EMTF Unpacker Update to Add HMT and Displaced Muon Information [12_3_X] #37465
EMTF Unpacker Update to Add HMT and Displaced Muon Information [12_3_X] #37465
Conversation
… following suggestion from Andrea Perrotta.
A new Pull Request was created by @eyigitba for CMSSW_12_3_X. It involves the following packages:
@epalencia, @cmsbuild, @cecilecaillol, @rekovic can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-02a8fc/23659/summary.html Comparison SummarySummary:
|
+l1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_4_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
backport of #37194 |
+1
|
PR description:
This PR updates the EMTF unpacker to add High Multiplicity Trigger (HMT) and Displaced Muon information. The EMTF firmware at P5 is already updated to add these parameters to EMTF DAQ output.
The main changes are in
EMTFBlockSP
for unpacking the new information. TheEMTFCollections
,EMTFSetup
EMTFTokens
andSP.h
are changed accordingly to include the new Regional Muon Shower object in the EMTF unpacker workflow.No changes are expected in performance.
PR validation:
Validated with
runTheMatrix.py -l limited -i all --ibeos
and no changes are observed.if this PR is a backport please specify the original PR and why you need to backport that PR:
This is a backport of PR #37194, to include this update in upcoming runs at P5.