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
Update final L1T menu tag for Run 3 start up #37442
Conversation
test parameters:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37442/29138
|
A new Pull Request was created by @francescobrivio for master. It involves the following packages:
@cmsbuild, @malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-688758/23620/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
Hmm the results seem interesting, in data where the new GT is used, there are differences in the tracking, see https://cmssdt.cern.ch/SDT/jenkins-artifacts/baseLineComparisons/CMSSW_12_4_X_2022-04-01-2300+688758/49366/139.001_RunMinimumBias2021+RunMinimumBias2021+HLTDR3_2021+RECODR3_MinBiasOffline+HARVESTD2021MB/ but the MCs that used the same new tag have no difference. Can @elfontan @silviodonato comment on this? Thanks |
Pinging also @Martin-Grunewald and @missirol who I know are involved in this development as well. |
Hi Silvio! The differences are due to "live" tags that got updated in the meanwhile (by O2Os or manually by users), but 139.001 is running on run 346512, so all these differences should be irrelevant. |
no idea... I'd suggest to run the tests one more time to confirm that the differences are real |
Before restarting the tests I'd like to see if there are other suggestions, because the backport PR (#37443 (comment)) tests are showing very similar results... |
They are very relevant instead. By updating the snapshot of the data GT you have unlocked this new IOV: which was announced here: https://cms-talk.web.cern.ch/t/offline-gt-updating-corrected-fixed-pixel-cpe-2d-template/8734/1 |
Hi @mmusich, indeed you are right, I had missed that IOV difference. Thanks a lot! |
+alca
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). 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) |
+1 |
PR description:
This PR updates the final L1T menu tag for Run 3 start up, as requested in this CMSTalk post, for Run3 MC GTs and the Run3 relval GTs used for TSG tests.
The new L1T tag is
L1Menu_Collisions2022_v1_0_0_xml
.GT differences:
Run 3 HLT RelVals
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_dataRun3_HLT_relval_v2/123X_dataRun3_HLT_relval_v3
Run 3 data RelVals
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_dataRun3_relval_v2/123X_dataRun3_relval_v3
2021 design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2021_design_v12/123X_mcRun3_2021_design_v13
2021 realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2021_realistic_v12/123X_mcRun3_2021_realistic_v13
2021 cosmics
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2021cosmics_realistic_deco_v12/123X_mcRun3_2021cosmics_realistic_deco_v13
2021 heavy ion
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2021_realistic_HI_v12/123X_mcRun3_2021_realistic_HI_v13
2023 realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2023_realistic_v12/123X_mcRun3_2023_realistic_v13
2024 realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun3_2024_realistic_v12/123X_mcRun3_2024_realistic_v13
Phase 2 realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/123X_mcRun4_realistic_v9/123X_mcRun4_realistic_v10
PR validation:
Tested with:
runTheMatrix.py -l 12034.0,11634.0,7.23,159.0,12434.0,12834.0 --ibeos -j16
Backport:
Not a backport, but a backport to 12_3_X will be provided soon.
FYI @elfontan @Martin-Grunewald