410 fix kokkos and kokkos kernels builds on ci#411
Conversation
Pipelines resultsmagistrate-build-amd64-ubuntu-20-04-clang-10-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-22-04-clang-13-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-20-04-clang-9-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-20-04-gcc-9-cuda-11-4-3-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-20-04-gcc-9-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-22-04-clang-12-cpp-vt Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-22-04-clang-11-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-20-04-gcc-9-cpp-docs Build for d49a291 (2026-01-13 23:04:29 UTC) magistrate-build-amd64-ubuntu-20-04-gcc-10-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-22-04-clang-15-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) magistrate-build-amd64-ubuntu-20-04-gcc-13-cuda-12-9-0-cpp Build for b27c86e (2026-01-14 20:29:59 UTC) magistrate-build-amd64-ubuntu-22-04-clang-14-cpp Build for be38b7a (2026-01-15 00:24:31 UTC) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #411 +/- ##
========================================
Coverage 96.52% 96.52%
========================================
Files 89 89
Lines 2823 2823
========================================
Hits 2725 2725
Misses 98 98
🚀 New features to boost your workflow:
|
|
@rppawlo It looks like Kokkos needs at least gcc 10.4 to work with CUDA 12. Thus, I'm creating a new pipeline here to support that configuration for testing: |
|
I think that github doesn't have GPU runners unless you pay for them :( |
f33b998 to
c35ddae
Compare
Needed to access map entries on device. Moved test to external function to call a device lambda. TEST_F body is in a private function and prohibits device code from being called there.
c35ddae to
3093505
Compare
|
@lifflander - is this close to being able to merge in? |
Yes I think we can merge |
|
Thanks @lifflander ! |
Fixes #410
Fixes #409
@nlslatt @rppawlo