-
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
[13_0_X] Add correct L1TCaloParams in 130X 2022 HI MC GT #41567
[13_0_X] Add correct L1TCaloParams in 130X 2022 HI MC GT #41567
Conversation
@cmsbuild please test |
A new Pull Request was created by @francescobrivio for CMSSW_13_0_X. It involves the following packages:
@tvami, @saumyaphor4252, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-58dbf9/32433/summary.html Comparison SummarySummary:
|
Just checking: no update to the 2023 HI GT? |
For the 2023 HI GT, the update was done at #41440. |
+alca
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_13_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Prompted by this comment and following the official request in this CMSTalk post, this PR updates the 2022 130X HI MC GT with the correct L1TCaloParams tag:
L1TCaloParams_static_HI_2022_v0_05_mc_v1
.This PR is a backport of #41437 and complements the other 13_0_X backport #41440.
GT difference:
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2022_realistic_HI_v5/130X_mcRun3_2022_realistic_HI_v6
PR validation:
Successfully run:
runTheMatrix.py -l 312.0 -j 8 --ibeos
Backport:
Backport of #41437