-
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
[12_3_X] Fix typo in L1T prescales files #37583
[12_3_X] Fix typo in L1T prescales files #37583
Conversation
backport of #37582 |
A new Pull Request was created by @francescobrivio 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
|
urgent
|
type bug-fix |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fa56ea/23951/summary.html Comparison SummarySummary:
|
merge |
PR description:
Backport of #37582
This PR fixes a typo introduced in #37453 by adding back the .xml string in the L1T prescale files.
The typo is a blocker for the correct operations of the L1T O2Os so it should be fixed asap.
PR validation:
Code compiles.
@cms-sw/l1-l2 please comment if further tests are needed on your side.
Backport:
Backport of #37582
FYI @dpiparo @cms-sw/db-l2 @panoskatsoulis @elfontan @boudoul