-
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
Add AlCaReco producers in phase-2 workflows #39858
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39858/32755
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@malbouis, @ChrisMisan, @yuanchao, @bbilin, @clacaputo, @cmsbuild, @AdrianoDee, @srimanob, @saumyaphor4252, @kskovpen, @sunilUIET, @tvami, @mandrenguyen, @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-64ceed/28531/summary.html Comparison SummarySummary:
|
Thank you for this, @mmusich ! I have created a versioned GT to replace the Candidate GT in autoCond.py: The only difference wrt to the previous GT is the AlCaRecoTriggerBits tag: https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/125X_mcRun4_realistic_v2/125X_mcRun4_realistic_v3 Would you please replace the Candidate Gt with this versioned one? |
+alca
|
941de35
to
638de99
Compare
+1 |
To clarify, the PR proposes to run ALCA step by default in all Phase-2 workflows. What is the cost, i.e. in terms of RelVals disk when we run full set of relvals, both NoPU and PU200? I see we currently do the same for Run-3 also. |
is not run. |
Ah, OK. The Alca step is not in
|
+Upgrade |
@cms-sw/pdmv-l2 please have a look and if you agree sign. |
+pdmv |
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, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
The goal of this PR is to start exercising Tracker alcareco producers in the phase-2 workflows, as there is interest in trying a preliminary alignment with the phase-2 geometry.
A minimal set of changes if proposed in order to achieve that goal, including the necessary update of the
phase2_realistic
autoCond key in order to get most updated trigger bits from the run-3 global tag (see also cmsTalk):In addition the pseudo-rapidity range of the alignment alcarecos is modified in the case of the phase-2 tracker to allow full coverage (up to |η|<4).
PR validation:
Run successfully
runTheMatrix.py -l 20834.0 -t 4 -j 8
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:
N/A
cc:
@consuegs