-
Notifications
You must be signed in to change notification settings - Fork 14
Description
We currently expose both the higher-level wrapper functions such as s2ftt.forward and s2fft.inverse and corresponding backend specific functions such as forward_jax, forward_numpy and forward_torch. This both makes our API surface larger than necessary, with multiple ways to do the same thing, and adds a level of redundancy as in having very similar docstrings across multiple functions. There are also various utility functions that are used mainly internally that are exposed as part of our API.
I would propose we expose a minimal public API of just the top-level wrapper functions and any utility functions we think are useful for users, and make all other functions undocumented / marked for internal use only by underscore prefixing the names. This would be a breaking change so would ideally be part of a major version bump.