Update GPU wfs for 1601 after renumbering in cmssw#2663
Update GPU wfs for 1601 after renumbering in cmssw#2663smuzaffar merged 1 commit intocms-sw:masterfrom
Conversation
|
A new Pull Request was created by @VourMa for branch master. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
enable gpu |
|
please test |
|
-1 Failed Tests: RelVals-AMD_W7900 Failed RelVals-AMD_W7900ValueError: Undefined workflows: 34634.712, 34634.713 Comparison SummarySummary:
|
|
I think this needs to be tested together with |
|
@VourMa there is a conflict to resolve now. |
e22052e to
686c996
Compare
|
Pull request #2663 was updated. |
Thanks for the heads-up, the conflict was now resolved. If the PR needs to be tested, I leave a note that this needs to be done in combination with cms-sw/cmssw#49984. |
|
test parameters:
|
|
enable gpu |
|
@cmsbuild, please test |
|
-1 Failed Tests: RelVals-AMD_MI300X Failed RelVals-AMD_MI300X
Comparison SummarySummary:
|
|
please test |
|
REMINDER @sextonkennedy, @mandrenguyen, @ftenchini: This PR was tested with cms-sw/cmssw#49984, please check if they should be merged together |
|
-1 Failed Tests: RelVals-NVIDIA_H100 Failed RelVals-NVIDIA_H100
Comparison SummarySummary:
|
|
This also needs cms-sw/cmssw#50039 to pass the tests. |
|
test parameters: pull_request = cms-sw/cmssw#49984, cms-sw/cmssw#50039 |
|
please test |
|
please test lets re-run tests. There was a bug in bot due to which bot pr changes were not properly merged during gpu tests, #2676 should fix that issue |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b29148/51189/summary.html Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
Max Memory Comparisons exceeding threshold NVIDIA_H100@cms-sw/core-l2 , I found 6 workflow step(s) with memory usage exceeding the error threshold: Expand to see workflows ...
|
|
@smuzaffar @mandrenguyen @ftenchini now that cms-sw/cmssw#49984 is merged, I think we should merge also this one. |
|
+externals |
|
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. @mandrenguyen, @ftenchini, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
cms-sw/cmssw#49832 updated the numbering of a few workflows to adhere to the conventions of Alpaka workflows. This PR updates the corresponding hardcoded list for the cms-bot tests to avoid failures.
Edit: The aforementioned PR has been superseded by cms-sw/cmssw#49984, which still needs this update.