-
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
Include ECAL Timing CC labelled tags in Run3 data and MC GTs #42958
Include ECAL Timing CC labelled tags in Run3 data and MC GTs #42958
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42958/37109
|
A new Pull Request was created by @saumyaphor4252 (Saumya Phor) for master. It involves the following packages:
@consuegs, @cmsbuild, @perrotta, @saumyaphor4252 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-ce8476/35066/summary.html Comparison SummarySummary:
|
based on these test results looks like this GT has to be updated to add the |
15b81fe
to
3a78801
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42958/37120
|
Pull request #42958 was updated. @saumyaphor4252, @cmsbuild, @consuegs, @perrotta can you please check and sign again. |
@cmsbuild, please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ce8476/35088/summary.html Comparison SummarySummary:
|
type bug-fix
Cc: @cms-sw/egamma-pog-l2 |
+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. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR is mostly a follow-up to #42928 and adds the Ecal Timing
CC
labelled tags toas follows:
For MC
For Data:
The relevant CMS Talk posts and links for the mentioned changes are [1][2][3]
[1] https://indico.cern.ch/event/1329876/
[2] https://cms-talk.web.cern.ch/t/queues-133x-mc-and-data-queues-opened/29683
[3] https://cms-talk.web.cern.ch/t/full-track-validation-ecal-time-calibration-conditions-for-cc-timing-algorithm/28247
In addition, it also introduces a new key
run3_data_PromptAnalysis
to theautoCond
. The corresponding GT is for the offline reprocessing happening with Prompt GTs. So far, the GT has conditions used for the 2022FG reprocessing and was last done in 130X cycle [4] with130X_dataRun3_PromptAnalysis_v1
. The difference between the 133X GT with the 130X GT used for nanov12 [4] is only the addition of the ECAL TimingCC
labelled tags. (See in the diffs below). No workflow is testing this GT for now. We leave that to PdMV to include the relevant workflows at some point.[4] https://cms-talk.web.cern.ch/t/mc-offline-rereco-gts-for-2022-remini-nano-v12-in-130x/29645
Apart from that it also does some minor changes:
GT Differences with the last ones are here:
Run3 data HLT : No change, just updated snapshot time
Run3 data Express : Includes Ecal Timing CC labelled tags, and updates as per the current Express GT with new snapshot time
Run3 data Prompt : Includes Ecal Timing CC labelled tags, and updates as per the current Prompt GT with new snapshot time
Run3 data Offline : Includes Ecal Timing CC labelled tags and updates the ES Channel status tag to fix the wrong IOV upload 375472 instead of 374572
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_dataRun3_v3/132X_dataRun3_v1
Run3 data PromptAnalysis : New key for offline reprocessing with Prompt GTs; Includes Ecal Timing CC labelled tags from the last GT used for NANOv12
130X_dataRun3_PromptAnalysis_v1
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_dataRun3_PromptAnalysis_v1/130X_dataRun3_PromptAnalysis_v1
Phase1 2022 design : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022_design_v1/132X_mcRun3_2022_design_v1
Phase1 2022 realistic : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022_realistic_v1/132X_mcRun3_2022_realistic_v2
Phase1 2022 realistic postEE : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022_realistic_postEE_v1/132X_mcRun3_2022_realistic_postEE_v2
Phase1 2022 cosmics : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022cosmics_realistic_deco_v1/132X_mcRun3_2022cosmics_realistic_deco_v2
Phase1 2022 cosmics design : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022cosmics_design_deco_v1/132X_mcRun3_2022cosmics_design_deco_v1
Phase1 2022 realistic hi : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2022_realistic_HI_v1/132X_mcRun3_2022_realistic_HI_v1
Phase1 2023 design : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023_design_v1/132X_mcRun3_2023_design_v1
Phase1 2023 realistic : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023_realistic_v1/132X_mcRun3_2023_realistic_v4
Phase1 2023 realistic postBPix : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023_realistic_postBPix_v1/132X_mcRun3_2023_realistic_postBPix_v3
Phase1 2023 cosmics reaslistic : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023cosmics_realistic_deco_v1/132X_mcRun3_2023cosmics_realistic_deco_v3
Phase1 2023 cosmics design : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023cosmics_design_deco_v1/132X_mcRun3_2023cosmics_design_deco_v1
Phase1 2023 realistic hi : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2023_realistic_HI_v1/132X_mcRun3_2023_realistic_HI_v2
Phase1 2024 realistic : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun3_2024_realistic_v1/132X_mcRun3_2024_realistic_v2
Phase2 : Includes Ecal Timing CC labelled tags
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/133X_mcRun4_realistic_v1/132X_mcRun4_realistic_v2
PR validation:
Tested successfully with the PR #42928 using
runTheMatrix.py -l 138.2,138.4,138.5,139.001,141.002,141.003,159.1,160.1,312.0,12034.0,12434.0,12834.0,11601.0,23234.0 -j10 --ibeos
runTheMatrix.py -l 12430.0,12440.0 --what upgrade -j 8 --ibeos
If 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:
Not a backport. Backport will be followed up for 132X and 131X to go with: