Skip to content

Link libcugraph dynamically against libcuvs#5499

Open
bdice wants to merge 2 commits intorapidsai:mainfrom
bdice:dynamic-libcuvs
Open

Link libcugraph dynamically against libcuvs#5499
bdice wants to merge 2 commits intorapidsai:mainfrom
bdice:dynamic-libcuvs

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Apr 23, 2026

Summary

  • remove the cuVS static-link configuration path and always link libcugraph against shared libcuvs
  • add libcuvs as a runtime dependency for conda and wheel builds, and exclude libcuvs.so from wheel bundling
  • re-enable the previously filtered CUDA test matrix entries now that libcugraph.so no longer carries the cuVS JIT LTO cudaLibrary* symbols

Validation

  • build-cugraph -j0
  • ldd cpp/build/conda/cuda-13.1/release/libcugraph.so shows libcuvs.so
  • nm -D --undefined-only cpp/build/conda/cuda-13.1/release/libcugraph.so | rg cudaLibrary returns no matches

Remove the cuVS static-link configuration path and make libcugraph depend on
libcuvs as a shared library for both conda and wheel builds. This moves the
cudaLibrary* runtime requirement out of libcugraph, restores the older CUDA
CI matrix entries, and keeps libcuvs ownership in the libcuvs package/wheel
instead of bundling it into libcugraph.
@bdice bdice requested review from a team as code owners April 23, 2026 15:34
@bdice bdice requested a review from AyodeAwe April 23, 2026 15:34
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Apr 23, 2026
@bdice bdice self-assigned this Apr 23, 2026
libcugraph now depends on shared libcuvs at runtime. Load libcuvs after
libraft and before libcugraph so wheel imports resolve the new dependency
chain consistently.
@bdice bdice requested a review from a team as a code owner April 24, 2026 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants