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

Splines kokkos kernels backend #480

Open
wants to merge 48 commits into
base: lapack-backend
Choose a base branch
from

Conversation

blegouix
Copy link
Collaborator

@blegouix blegouix commented Jun 5, 2024

No description provided.

@blegouix blegouix self-assigned this Jun 5, 2024
@blegouix blegouix changed the base branch from main to lapack-backend June 5, 2024 14:31
@blegouix blegouix marked this pull request as draft June 5, 2024 14:31
@blegouix blegouix changed the title [Experimental] Splines kokkos kernels backend Splines kokkos kernels backend Jun 19, 2024
@tpadioleau tpadioleau force-pushed the kokkos-kernels-backend branch 2 times, most recently from d8e99bd to 59c7a13 Compare June 25, 2024 21:09
@tpadioleau
Copy link
Member

@yasahi-hpc If you have time can you have a look at the following error

[  6%] Building CXX object examples/CMakeFiles/characteristics_advection.dir/characteristics_advection.cpp.o
In file included from /data/opt/kokkos-kernels/include/KokkosBatched_Gbtrs.hpp:62,
                 from /src/include/ddc/kernels/splines/splines_linear_problem_band.hpp:19,
                 from /src/include/ddc/kernels/splines/splines_linear_problem_maker.hpp:12,
                 from /src/include/ddc/kernels/splines/spline_builder.hpp:15,
                 from /src/include/ddc/kernels/splines.hpp:17,
                 from /src/examples/characteristics_advection.cpp:12:
/data/opt/kokkos-kernels/include/KokkosBatched_Gbtrs_Serial_Impl.hpp:21:10: fatal error: ../src/KokkosBatched_Tbsv.hpp: No such file or directory
   21 | #include "../src/KokkosBatched_Tbsv.hpp"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

My guess is that the path is not valid when the library gets installed ?

@yasahi-hpc
Copy link
Member

@tpadioleau
I made a fix. Could you please test again

@tpadioleau
Copy link
Member

tpadioleau commented Jun 26, 2024

@tpadioleau I made a fix. Could you please test again

What branch are we based on ?

I guess @blegouix just did the update but I am still interested in the answer

@blegouix
Copy link
Collaborator Author

blegouix commented Jun 26, 2024

@tpadioleau I made a fix. Could you please test again

What branch are we based on ?

I guess @blegouix just did the update but I am still interested in the answer

develop-spline-kernels from the Yuuichi fork.

@yasahi-hpc
Copy link
Member

Failures. I guess I need to include KokkosBatched_Tbsv.hpp in KokkosBatched_Gbtrs_Serial_Impl.hpp

@tpadioleau
Copy link
Member

Failures. I guess I need to include KokkosBatched_Tbsv.hpp in KokkosBatched_Gbtrs_Serial_Impl.hpp

I let you see about it, if you get a circular dependency you can also use forward declarations.

@yasahi-hpc
Copy link
Member

It should be OK now.

I let you see about it, if you get a circular dependency you can also use forward declarations.

We do not have a circular dependency here. I just included a wrong file

@tpadioleau
Copy link
Member

all green!

CMakeLists.txt Outdated Show resolved Hide resolved
@blegouix
Copy link
Collaborator Author

blegouix commented Jul 1, 2024

@tpadioleau should I rename ddc::SplineSolver::LAPACK to ddc::SplineSolver::KokkosKernels ?

@tpadioleau
Copy link
Member

tpadioleau commented Jul 2, 2024

@tpadioleau should I rename ddc::SplineSolver::LAPACK to ddc::SplineSolver::KokkosKernels ?

It is ok for now, let's keep LAPACK

@blegouix
Copy link
Collaborator Author

blegouix commented Jul 2, 2024

@yasahi-hpc I revert your commit after our discussion, we do not need push/pop with nvtx-connector

@blegouix blegouix marked this pull request as ready for review July 2, 2024 16:39
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

3 participants