-
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
fix GEM inputs for L1REPACK:FullMC
[12_4_X
]
#40470
fix GEM inputs for L1REPACK:FullMC
[12_4_X
]
#40470
Conversation
A new Pull Request was created by @missirol (Marino Missiroli) for CMSSW_12_4_X. It involves the following packages:
@cmsbuild, @perrotta, @rappoccio, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
type bugfix This fix alone does not warrant a new release (users can locally merge this PR, if needed). At the same time, I think it would be good to merge it to ease trigger studies. |
FYI: @cms-sw/l1-l2 (Feel free to "assign l1", but I don't think it's necessary given that the equivalent #40461 was signed by L1T.) |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3c7b20/29898/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_0_X is complete. This pull request will be automatically merged. |
backport of #40461
(verbatim backport)
PR description:
This PR fixes the
L1REPACK:FullMC
step (full simulation of trigger primitives + emulation of L1T from RAW in MC events [ref]), where GEM inputs are currently misconfigured. This type of configuration is used in offline trigger studies, even though it is untested in CMSSW.For further details, please the description of the original PR, i.e. #40461.
Small edits are applied to other L1REPACK
cff
s, to fix a few print-outs (see #40461 (comment) for details).PR validation:
None beyond the checks done for #40461.
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:
#40461
Trigger studies in
12_4_X
.