-
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
Update a few MC Gts in autoCond to the latest version available on 14/10/2024 #46379
Update a few MC Gts in autoCond to the latest version available on 14/10/2024 #46379
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46379/42223 |
A new Pull Request was created by @perrotta for master. It involves the following packages:
@atpathak, @cmsbuild, @consuegs, @perrotta can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
please test for CMSSW_14_2_DBG_X |
I would be curious to understand why 2024 MC workflows show differences in tracking (e.g. generalTracks in wf 12834.0) when the only difference between the GTs used for them in reference and PR produced samples is the EcalLaserAPDPNRatios tag, see here the diff wrt the 2024MC GT in autoCond in the reference IB. If someone has an explanation I would be happy to hear about it. On the other hand, since the GT update is quite simple and clear, I would be tempted to approve this PR nonetheless. |
differences look widespread and compatible with a randomizing effect from regenerated random number sequences in the digitization step. I see that |
It makes sense, indeed. Thank you Marco. |
+1
|
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. @sextonkennedy, @mandrenguyen, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
The following GTs are updated in autoCond (in some cases they are not yet the final ones, but they can be tested as such in the IBs).
phase1_2022_realistic
: 140X_mcRun3_2022_realistic_v11phase1_2023_realistic
: 140X_mcRun3_2023_realistic_v7phase1_2024_realistic
: 140X_mcRun3_2024_realistic_v26EcalLaserAPDPNRatios_2024_outliers_cleaned_3sigma_mc
with the correctEcalLasrtAPDPNRatiosRcd
(while it had been previously wrongly uploaded in theEcalLaserAPDPNRatiosMCRcd
)phase1_2024_realistic_hi
: 141X_mcRun3_2024_realistic_HI_v8phase1_2024_realistic_ppRef5TeV
: 141X_mcRun3_2024_realistic_ppRef5TeV_v5EcalLaserAPDPNRatios_2024_outliers_cleaned_3sigma_mc
with the correctEcalLasrtAPDPNRatiosRcd
(while it had been previously wrongly uploaded in theEcalLaserAPDPNRatiosMCRcd
)phase2_realistic
: 141X_mcRun4_realistic_v3Backports to CMSSW_14_1_X and CMSSW_14_0_X (limitately to the relevant GTs) wil be provided