-
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
PPS renaming alca reco producers - backport of PR#36702 to 12_2_X #36784
PPS renaming alca reco producers - backport of PR#36702 to 12_2_X #36784
Conversation
PromptCalibProdPPS -> PromptCalibProdPPSTimingCalib
Via #58 * renaming according to guidelines for Tier0 replay * pps alignment renaming - part 2 * pps alignment renaming - part 3 (paths) * Updated the PPS alignment AlCa file names. PPSAlignment -> PromptCalibProdPPSAlignment Co-authored-by: Mateusz <mateuszkocot99@gmail.com>
Conflicts: Configuration/AlCa/python/autoPCL.py Configuration/EventContent/python/AlCaRecoOutput_cff.py Configuration/StandardSequences/python/AlCaRecoStreams_cff.py
A new Pull Request was created by @grzanka (Leszek Grzanka) for CMSSW_12_2_X. It involves the following packages:
@perrotta, @malbouis, @yuanchao, @jordan-martins, @bbilin, @wajidalikhan, @tvami, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen, @qliphy, @francescobrivio, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
@cmsbuild please test |
backport of #36702 |
-1 Failed Tests: RelVals-INPUT The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: RelVals-INPUT
Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+Upgrade Backport PR. PR test runs fine, the additional workflows with the change run fine. |
+alca |
Hi @kskovpen you signed the original PR for this, please consider signing the backport as well, thanks! |
+1 |
@tvami This week's ORP will be cancelled due to ongoing CMSWeek. However, we can merge this PR and build next 12_2_X this week if it is really urgently needed. |
merge |
Yes that would be great! But we can wait for the geometry backports before cutting a new 12_2_X release. |
PR description:
This is a backport of #36702 to 12_2_X
It is related to cms-AlCaDB/AlCaTools#53
The changes are necessary for T0 replay with PPS. The suggestions for renaming appeared in the discussion of #36273
It is following #36698
PR validation:
Tested with: