-
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 all GPU workflows [12.3.x] #37413
Update all GPU workflows [12.3.x] #37413
Conversation
backport #37411 |
enable gpu |
please test |
A new Pull Request was created by @fwyzard (Andrea Bocci) for CMSSW_12_3_X. It involves the following packages:
@jordan-martins, @bbilin, @wajidalikhan, @AdrianoDee, @srimanob, @kskovpen can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-275236/23528/summary.html GPU Comparison SummarySummary:
Comparison SummarySummary:
|
73d725f
to
ce086a6
Compare
Pull request #37413 was updated. @jordan-martins, @bbilin, @wajidalikhan, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please check and sign again. |
please test |
Add Pixel-only and HCAL-only validation workflows. Add a single workflow running all GPU-enabled reconstruction. Do not customise the HLT, as it already makes full use of GPU reconstruction when the "gpu" modifier is enabled. Add a short description before each workflow.
ce086a6
to
187ef7e
Compare
Pull request #37413 was updated. @jordan-martins, @bbilin, @wajidalikhan, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please check and sign again. |
please test |
test parameters:
|
please test |
@smuzaffar did the last command request rerunning all the tests, or only the new one on gpu ? |
@fwyzard , it runs all default tests/workflows plus extra test mentioned in the |
ah... OK, sorry for the duplicate work :-/ |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-275236/23571/summary.html GPU Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
Comparison SummarySummary:
|
+Upgrade Backport PR. |
urgent |
kindly ping @cms-sw/pdmv-l2 |
+pdmv |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_3_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_4_X is complete. 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) |
+1
|
PR description:
Add Pixel-only and HCAL-only validation workflows:
#.503
(quadruplets),#.507
(triplets)#.523
Enable the existing Pixel-only and HCAL-only profiling workflows:
#.504
(quadruplets),#.508
(triplets)#.524
Add a single workflow running all GPU-enabled reconstruction (Pixel, ECAL, HCAL):
#.581
(quadruplets) and#.585
(triplets)#.582
(pixel quadruplets) and#.586
(triplets)#.583
(quadruplets) and#.587
(triplets)Do not customise the HLT, as it already makes full use of GPU reconstruction when the "gpu" modifier is enabled.
Add a short description before each workflow.
PR validation:
All new or updated GPU workflows ran successfully in
CMSSW_12_4_X_2022-03-23-2300
:Pixel-only
ECAL-only
HCAL-only
All GPU sequences
Full offline reconstruction with all GPU sequences
if this PR is a backport please specify the original PR and why you need to backport that PR:
Backport of #37411.