-
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
[13_0_X] Update GEM geometry in 2023 MC GTs and the HI beamspot in 2023 MC GT #41203
[13_0_X] Update GEM geometry in 2023 MC GTs and the HI beamspot in 2023 MC GT #41203
Conversation
A new Pull Request was created by @saumyaphor4252 (Saumya Phor) for CMSSW_13_0_X. It involves the following packages:
@cmsbuild, @tvami, @saumyaphor4252, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
@cmsbuild, please abort |
1a247d5
to
1db7f4d
Compare
Pull request #41203 was updated. @cmsbuild, @tvami, @saumyaphor4252, @francescobrivio can you please check and sign again. |
@cmsbuild, please test |
-1 Failed Tests: RelVals RelVals-INPUT AddOn RelVals
RelVals-INPUT
AddOn Tests
Expand to see more addon errors ... |
1db7f4d
to
7757cd1
Compare
Pull request #41203 was updated. @cmsbuild, @tvami, @saumyaphor4252, @francescobrivio can you please check and sign again. |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4bb521/31639/summary.html Comparison SummarySummary:
|
This is mostly in 12434.0 which is where we expect changes. They are widespread, which is also expected for the geometry change |
+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_1_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) |
urgent
|
backport of #41202 |
+1
|
PR description:
Backport of #41202
This PR updates the
130X
2023 MC Global Tags inautoCond
with the GEM geometry tags requested in this CMS Talk post [1]. Also, the 2023 Heavy Ion MC GT is updated with the beamspot tagBeamSpotObjects_Realistic2022PbPbCollision_v5_mc
as requested in [2].[1] https://cms-talk.web.cern.ch/t/update-of-gem-geometry-for-2023-data-and-mc/21887
[2] https://cms-talk.web.cern.ch/t/request-for-new-run3-hi-mc-gt-realistic-bs-for-13-0-x/21912
The difference with the last GTs in
autoCond
is here:https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2023_design_v3/130X_mcRun3_2023_design_v5
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2023cosmics_realistic_deco_v3/130X_mcRun3_2023cosmics_realistic_deco_v5
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2023cosmics_design_deco_v3/130X_mcRun3_2023cosmics_design_deco_v5
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2023_realistic_v4/130X_mcRun3_2023_realistic_v6
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/130X_mcRun3_2023_realistic_HI_v4/130X_mcRun3_2023_realistic_HI_v7
PR validation:
GTs tested locally with
runTheMatrix.py -l 160.1, 12434.0 --ibeos -j 16
which consumeauto:phase1_2023_realistic_hi
andauto:phase1_2023_realistic
respectivelyIf 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:
Partial backport of #41202. No other backport foreseen.