Make pyssht and healpy optional dependencies #267
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related to #229
Removes
pyssht
andhealpy
from required dependencies fors2fft
as these are currently only used in tests and in JAX wrapper functions ins2fft.transforms.c_backend_spherical
. Instead they are now added as test specific dependencies and thes2fft.transforms.c_backend_spherical
module updated to try to import these modules when calling the wrapper functions and raise an exception with an informative error message instructing the user they need to install the relevant package to use the wrapper if not already installed.As the dependency on
pyssht
is what was requiring pinning NumPy to<2
this resolves #266 ass2fft
can then be installed with NumPy v2 or above if not installingpyssht
, although here we still pin the NumPy version in the test dependencies so we are not testing against NumPy v2.