-
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 helper functions for alpaka kernels [12.5.x] #39547
Conversation
A new Pull Request was created by @fwyzard (Andrea Bocci) for CMSSW_12_5_X. It involves the following packages:
@cmsbuild, @makortel, @fwyzard can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
backport #39546 |
enable gpu |
please test with cms-sw/cmsdist#8107 |
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77b26/27869/summary.html |
please test with cms-sw/cmsdist#8108 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77b26/27871/summary.html Comparison SummarySummary:
GPU Comparison SummarySummary:
|
58de740
to
05286d1
Compare
please test with cms-sw/cmsdist#8108 |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test testHeterogeneousCoreAlpakaTestWriteRead had ERRORS Comparison SummarySummary:
GPU Comparison SummarySummary:
|
Add helper functions for: - performing an integer division with rounding up, or rounding a value to the next multiple of an integer; - creating an efficient work division, assigning the number of work items either to the number of GPU threads or CPU elements; - perform a scalar or N-dimensional strided loop within a kernel, based on the grid size and number of elements.
de68e6d
to
355d9d3
Compare
enable gpu |
please test |
+heterogeneous |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_5_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_12_6_X is complete. 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) |
I assume the |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77b26/27956/summary.html Comparison SummarySummary:
GPU Comparison SummarySummary:
|
+1 |
PR description:
Add helper functions for:
Update the alpaka tests to use the new helper functions.
PR validation:
Unit tests pass.
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:
Backport of #39546 to CMSSW_12_5_X for the Alpaka migration.