Update alpaka to 2.0.0 "Concepts Ahead"#10103
Update alpaka to 2.0.0 "Concepts Ahead"#10103cmsbuild merged 1 commit intocms-sw:IB/CMSSW_16_0_X/masterfrom
Conversation
|
A new Pull Request was created by @fwyzard for branch IB/CMSSW_16_0_X/master. @akritkbehera, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
enable gpu |
|
please test with cms-sw/cmssw#49042 |
|
-1 Failed Tests: UnitTests RelVals-NVIDIA_T4 nvidia_t4UnitTests 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: Unit TestsI found 2 errors in the following unit tests: ---> test alpakaTestOneHistoContainerSerialSync had ERRORS ---> test alpakaTestPrefixScanSerialSync had ERRORS RelVals-NVIDIA_T4
Expand to see more relval errors ...Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
|
|
please test with cms-sw/cmssw#49042 |
|
-1 Failed Tests: UnitTests Unit TestsI found 2 errors in the following unit tests: ---> test alpakaTestOneHistoContainerSerialSync had ERRORS ---> test alpakaTestPrefixScanSerialSync had ERRORS Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
NVIDIA_T4 Comparison SummarySummary:
|
fwyzard
force-pushed
the
IB/CMSSW_16_0_X/master_alpaka_2.0
branch
from
17b606b to
eaf0170
Compare
|
Pull request #10103 was updated. |
|
please test with cms-sw/cmssw#49042 |
|
@smuzaffar cms-sw/cmssw#49042 is required to update CMSSW for alpaka 2.0 |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-dc1302/48883/summary.html 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: Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
NVIDIA_T4 Comparison SummarySummary:
|
|
@Parsifal-2045 (thanks!) run timing measurement using the |
|
Thanks for the check 👍🏻 |
|
please test with cms-sw/cmssw#49042 for el8_aarch64_gcc13 |
| @@ -1,5 +1,4 @@ | |||
| <tool name="@TOOL@" version="@TOOL_VERSION@" revision="2"> | |||
| <use name="boost"/> | |||
There was a problem hiding this comment.
@fwyzard , can you please update the revision here?
There was a problem hiding this comment.
Can you explain how it works ?
I understand updating the revision if there are only changes to the xml file, but if I already update the version, isn't that enough ?
In fact, wouldn't it make sense to reset the revision to 1 when one updates the version ?
There was a problem hiding this comment.
revision is kind of version of the xml file, so when ever you change the contents of xml file then update the revision too.
There was a problem hiding this comment.
OK.
I disagree, but will comply.
|
due to max commit status limit reached for this PR, so for now I have disabled bot to process it. Once we have a new commit pushed to this branch then I will re-enable bot to keep on processing this PR |
|
rebased without changes to get a new commit |
| @@ -1,5 +1,4 @@ | |||
| <tool name="@TOOL@" version="@TOOL_VERSION@" revision="2"> | |||
There was a problem hiding this comment.
| <tool name="@TOOL@" version="@TOOL_VERSION@" revision="2"> | |
| <tool name="@TOOL@" version="@TOOL_VERSION@" revision="3"> |
|
@fwyzard , due to commit status limit bot was not able to run the last PR tests. Please apply the suggested change https://github.com/cms-sw/cmsdist/pull/10103/files#r2477981803 (which will create a new commit/different commit and reset the commit statuses) so that bot can continue processing this PR |
This version introduces C++20 compatibility and removes the dependency on Boost. See https://github.com/alpaka-group/alpaka/releases/tag/2.0.0 for the release notes, and https://github.com/alpaka-group/alpaka/pulls?q=milestone%3A2.0.0 for the full list of changes in the 2.0.0 release.
|
please test |
|
Pull request #10103 was updated. |
|
test parameters: |
|
please test |
|
please test for el8_aarch64_gcc13 |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-dc1302/49102/summary.html |
|
-1 Failed Tests: RelVals-NVIDIA_T4 RelVals-NVIDIA_T4
Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
|
|
The test is already failing without changes on top of the IB, see for example https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-61b8db/49103/summary.html |
|
ignore tests-rejected with ib-failure |
|
@mandrenguyen @smuzaffar can we merge this from 16.0.0-pre2 ? |
|
@fwyzard If @cms-sw/externals-l2 signs then I'll merge. |
|
+externals |
|
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_16_0_X/master IBs (test failures were overridden). This pull request will now be reviewed by the release team before it's merged. @ftenchini, @mandrenguyen, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
|
Thanks ! |
6 participants
This version introduces C++20 compatibility and removes the dependency on Boost.
See here for the release notes, and here for the full list of changes in the 2.0.0 release.