Open DrTimothyAldenDavis opened 10 months ago
I guess it's because the rmm_wrap sub-package doesn't have the new include ( CMakePackageConfigHelpers )
and following, which all the other packages have?
One option, which I'll have to bounce off of my NVIDIA collaborators, is to merge rmm_wrap into the GraphBLAS_CUDA package. That way I reduce the # of packages by one.
I'm looking into it. Should probably be not too difficult.
See #543.
Fixed -- thanks!
One option, which I'll have to bounce off of my NVIDIA collaborators, is to merge rmm_wrap into the GraphBLAS_CUDA package. That way I reduce the # of packages by one.
If the approach in #547 turns out to work, we could probably do the same for rmm_wrap and absorb it into GraphBLAS_CUDA. Do we need to wait for approval from NVIDIA for that?
That would be a good idea; it would simplify the linking of the packages for the end user application.
I'll double-check with NVIDIA. Since the CUDA acceleration for GraphBLAS isn't ready for production anyway, let's wait until after I post a stable SuiteSparse 7.4.0, with rmm_wrap as-is.
@mmuetzel : I tried to enable cuda in SuiteSparse/GraphBLAS by setting ENABLE_CUDA to ON in SuiteSparse/GraphBLAS/CMakeLists.txt, but the cmake reports an error about not finding rmm_wrap. I realize that you can't test the CUDA case on your end, but here's what I'm seeing. Is this something that needs to be added to GraphBLAS/rmm_wrap/CMakeLists.txt?