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
pysshtandhealpyfrom required dependencies fors2fftas 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_sphericalmodule 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
pysshtis what was requiring pinning NumPy to<2this resolves #266 ass2fftcan 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.