-
Notifications
You must be signed in to change notification settings - Fork 218
fix: Make GNN plugin buildable without CUDA #4631
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
base: main
Are you sure you want to change the base?
Conversation
For completeness: this allows to build with the following combination of flags:
|
Thanks for adding this! I did not look into CPU only builds for years now... In principle we build this combination in the CI I think, but there we do have CUDA in the image and actually augment the path to find it... Maybe you could try if you can remove this line from the CI? Line 115 in 342ea6b
|
Seems to be the appropriate thing to do here, since FetchContent_MakeAvailable doesn't work for the use case here.
0e0f9ae
to
ea338c5
Compare
ea338c5
to
4bfe693
Compare
|
Guard FRNN building and linking behind more CMake conditions and fix a few compiler warnings
--- END COMMIT MESSAGE ---
@benjaminhuth these are a few minor things I stumbled over when trying to build entirely without CUDA.