-
Notifications
You must be signed in to change notification settings - Fork 15
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
Move main array namespace #307
Comments
Either seems fine. (2) might be slightly better, because it would follow EDIT: On the other hand |
I went through the same thought process: |
Another very good explanation of the decision to move away from the separate |
That's really interesting. Related: I would like to test cubed(/pint/sparse/every other array library) by using hypothesis.extra.array_api.make_strategies_namespace, which requires passing in the array-API-compliant namespace directly. See these docs (just released in the latest version of xarray) for an explanation of what I'm talking about. |
Currently all the array functions are in
cubed.array_api
. This was created to follow the naming pattern for the new array API innumpy.array_api
. Since then, however, there has been a change in emphasis to viewnumpy.array_api
as a reference implementation so downstream libraries can test if they are using only array API functions - it is no longer recommended for end users to use it directly, since array libraries are slowly migrating their main namespaces to implement the array API. This includes NumPy in version 2.0, and in the meantime there is array-api-compat to ease migration for users of NumPy, CuPy, and PyTorch.The upshot for Cubed is that we should probably move away from
cubed.array_api
for the above reasons, but also so we can add new functions that are not yet standardised, such asnansum
(see #153).The two options I see are:
cubed
cubed.array
Thoughts?
The text was updated successfully, but these errors were encountered: