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
Remove L1Reco from Phase2 workflow #38010
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38010/30090
|
A new Pull Request was created by @srimanob (Phat Srimanobhas) for master. It involves the following packages:
@jordan-martins, @bbilin, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen 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-79c5f8/24851/summary.html Comparison SummarySummary:
|
+pdmv |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-79c5f8/25046/summary.html Comparison SummarySummary:
|
+Upgrade This PR is to remove obsolete L1Reco step from Phase-2 sequences. No change is expected as no one in Phase-2 should use objected produced from L1Reco. Additional L1 steps will be added to RECO sequence if need, in later PR. This is in discussion with L1T. |
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) |
@srimanob would this removal affect validation and monitoring of the L1T developments for phase 2 which are currently being integrated in CMSSW? |
+1
|
PR description:
As discussed in #33012, the L1Reco step is obsolete for Phase-2. It is an unclear point if we should rerun
L1TrackTrigger,L1
again inRECO
or not. This is to be clarified.PR validation:
None
if this PR is a backport please specify the original PR and why you need to backport that PR:
No need of backport.