You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have lots of functions, like ak.sum and ak.concatenate, but we shouldn't be cluttering the namespace with things like ak.ak_sum and ak.ak_concatenate:
These two sets can be removed without fanfare. (If any users are accessing ak.sum as ak.ak_sum.sum, they should be corrected!)
We should probably also get rid of the visibility of "stepping stone" modules like ak.operations, although this one would require a deprecation cycle because it has been used in our tests, and some users might have adopted it for that reason. It can be deprecated by moving operations to _operations and creating a new operations that provides everything through a __getattr__ (PEP-562) that raises a warning.
Ideally, there should be only one correct path to each public API function.
We have lots of functions, like
ak.sum
andak.concatenate
, but we shouldn't be cluttering the namespace with things likeak.ak_sum
andak.ak_concatenate
:These two sets can be removed without fanfare. (If any users are accessing
ak.sum
asak.ak_sum.sum
, they should be corrected!)We should probably also get rid of the visibility of "stepping stone" modules like
ak.operations
, although this one would require a deprecation cycle because it has been used in our tests, and some users might have adopted it for that reason. It can be deprecated by movingoperations
to_operations
and creating a newoperations
that provides everything through a__getattr__
(PEP-562) that raises a warning.Ideally, there should be only one correct path to each public API function.
It would also end up removing this:
awkward/src/awkward/__init__.py
Lines 52 to 53 in f9a29ef
The text was updated successfully, but these errors were encountered: