flatironinstitute / jax-finufft

JAX bindings to the Flatiron Institute Non-uniform Fast Fourier Transform (FINUFFT) library
Apache License 2.0
80 stars 3 forks source link

Updating CUDA build to be opt in #43

Closed dfm closed 11 months ago

dfm commented 11 months ago

I was running into issues where even when a CUDA compiler was found, compilation would still fail because of missing include directories, and we didn't have a way to turn off CUDA compilation. This PR updates the build infrastructure to only compile the CUDA extensions when they are explicitly requested using -DJAX_FINUFFT_USE_CUDA=ON. It should be sufficient to set a CMAKE_ARGS environment variable when running pip install.

Still to do: Update the README.