Skip to content
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

Fix Kokkos kernels for Kokkos 4.3.0 #641

Open
wants to merge 1 commit into
base: default
Choose a base branch
from
Open

Conversation

caschb
Copy link

@caschb caschb commented Apr 5, 2024

Kokkos kernels don't compile under the latest release. Math functions such as fabs have been moved from the Kokkos::Experimental namespace to the Kokkos namespace. Additionally, directly including headers aside from Kokkos_Core.hpp is disallowed.

#640

Do you certify that your contribution is made in good faith and does not attempt to introduce any negative behavior into this project?

  • Yes
  • No

Kokkos kernels don't compile under the latest release.
Math functions such as fabs have been moved from the Kokkos::Experimental namespace to the Kokkos namespace.
Additionally, directly including headers aside from Kokkos_Core.hpp is disallowed.
@caschb caschb requested a review from jeffhammond as a code owner April 5, 2024 20:17
@jeffhammond
Copy link
Member

It's too bad Travis CI went away and regressions aren't caught anymore. I haven't had time to do GitHub CI yet.

Anyways, I'll verify the change and merge soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants