Skip to content

RFC: add support for a tuple of axes in expand_dims #760

Closed

Description

@izaid

Hello all! I raised this issue on array-api-compat earlier (data-apis/array-api-compat#105), but I think it might be more properly directed here.

In the array API, expand_dims supports only a single axis (https://data-apis.org/array-api/latest/API_specification/generated/array_api.expand_dims.html) as opposed to a tuple of axes. This is different to NumPy, CuPy, and Jax, which support a tuple of axes. PyTorch, however, supports only a single axis. I don't know the justification for why the array API supports only a single axis as opposed to a tuple, but what it means is that expand_dims no longer works in many places when adopting the array API.

In practice, expand_dims is just a light wrapper for reshape, see https://github.com/numpy/numpy/blob/3b246c6488cf246d488bbe5726ca58dc26b6ea74/numpy/lib/_shape_base_impl.py#L594. But it's not great to force users to write their own version of expand_dims in every library now. Is the array API willing to update expand_dims to support a tuple of axes? If not, and if expand_dims will only support a single axis going forward, that effectively makes all users of expand_dims copy and paste the NumPy implementation.

@lucascolley Pointed out to me that when expand_dims was added to the array API, only NumPy supported a tuple of axes. See #42. That was 4 years ago and the situation has changed, as above.

Activity

  1. asmeurer commented on Mar 11, 2024

    @asmeurer
    Member

    Seems tuple support was omitted because torch doesn't support it #42. I found a few feature requests for it for torch.unsqueeze (the PyTorch equivalent to expand_dims) pytorch/pytorch#30702, pytorch/pytorch#4692 (comment). Seems it was intentionally omitted due to the ambiguity that arises from mixing negative and positive indices.

    I agree this ambiguity is a potential concern. If we standardize this, we should somehow only require a subset of behavior that omits this ambiguity, e.g., by leaving the mixing of negative and nonnegative indices unspecified.

    Consider for example:

    >>> np.expand_dims(np.empty((2,)), (1, -1)).shape
    (2, 1, 1)

    The resulting shape has 1 in positions 1 and -1, but a result shape of (2, 1) would also satisfy this. I suppose one could argue that exactly len(axes) dimensions should be added.

    But also consider

    >>> np.expand_dims(np.empty((2, 3, 4, 5)), (3, -3)).shape
    Traceback (most recent call last):
      File "<stdin>", line 1, in <module>
      File "/Users/aaronmeurer/miniconda3/envs/array-apis/lib/python3.11/site-packages/numpy/lib/shape_base.py", line 597, in expand_dims
        axis = normalize_axis_tuple(axis, out_ndim)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/Users/aaronmeurer/miniconda3/envs/array-apis/lib/python3.11/site-packages/numpy/core/numeric.py", line 1385, in normalize_axis_tuple
        raise ValueError('repeated axis')
    ValueError: repeated axis

    There's no way to insert 1 dimensions into (2, 3, 4, 5) so that they appear at indices 3 and -3.

    Here's a small proof. There's no length list where you can remove indices 3 and -3 and result in a list of length 4
    >>> def remove_indices(n, idxes):
    ...     """Return range(n) with `idxes` indices removed"""
    ...     x = list(range(n))
    ...     vals = [x[i] for i in idxes]
    ...     for v in vals:
    ...         try:
    ...             x.remove(v)
    ...         except ValueError: # Already removed
    ...             pass
    ...     return x
    >>> [remove_indices(n, (-3, 3)) for n in range(4, 10)]
    [[0, 2], [0, 1, 4], [0, 1, 2, 4, 5], [0, 1, 2, 5, 6], [0, 1, 2, 4, 6, 7], [0, 1, 2, 4, 5, 7, 8]]
    >>> [len(remove_indices(n, (-3, 3))) for n in range(4, 10)]
    [2, 3, 5, 5, 6, 7]

    At the same time, if the goal of expand_dims is for the axes to refer to the dimensions after unsqueezing/expanding, then it's not exactly trivial to do it as a sequence of expand_dims, because if you apply the expansion in the wrong order you will break the position of previous dimensions (the correct logic is not hard, but it's the sort of thing that's easy to get wrong). So I think there is value in having native support for multiple axes.

  2. changed the title [-]`expand_dims` for tuple of axes[/-] [+]RFC: add support for a tuple of axes in `expand_dims`[/+] on Apr 4, 2024
  3. added
    RFCRequest for comments. Feature requests and proposed changes.
    topic: ManipulationArray manipulation and transformation.
    on Apr 4, 2024
  4. added this to the v2024 milestone on Apr 4, 2024
  5. Micky774 commented on Apr 15, 2024

    @Micky774
    Contributor

    Regarding removing ambiguity, I think it would suffice to impose an ordering in which to prefer expanding dims right? For example, if we specify "negative indices get resolved first" then your borrowing your example above could be resolved as

    x = np.empty((2, 3, 4, 5))
    xp.expand_dims(x., (3, -3)) == np.expand_dims(np.expand_dims(x, -3), 3)

    so that the final output shape is (2, 3, 1, 1, 4, 5), which seems reasonable.

    Still, I'm not sure if it is worth it since in the first place users could do it in a two-step expansion (albeit with some more thought), and the resolution order (+ or - indices first?) is rather arbitrary.

  6. asmeurer commented on Apr 17, 2024

    @asmeurer
    Member

    When you do repeated expand_dims, the inserted dimensions in the final shape won't necessarily be in the indices you initially specified (that's the whole point of this feature request, that you need a way to do them all at once). (2, 3, 1, 1, 4, 5) has 1s at indices 2 and -3 (remember 0-based indexing), because the 1 that was at index -3 got shifted over.

  7. lucascolley commented on Dec 23, 2024

    @lucascolley
    Member

    in case it affects this making v2024 either way, this is now available as https://data-apis.org/array-api-extra/generated/array_api_extra.expand_dims.html

  8. kgryte commented on Jan 20, 2025

    @kgryte
    Contributor

    Given the ambiguity of supporting a tuple of axes in expand_dims, I wonder if there is room for an alternative API which avoids the ambiguity altogether. Namely,

    def spread_dims(x: array, ndims: int, axes=Tuple[int, ...]) -> array

    which expands the shape of an input array x to have ndims and where the current dimensions are explicitly mapped to a unique list of axes in the resulting array. All unspecified axes must be singleton dimensions.

    This essentially flips the problem into one in which you specify where you want the non-singleton dimensions, rather than where you want to insert the singleton dimensions.

  9. lucascolley commented on Jan 20, 2025

    @lucascolley
    Member

    That sounds like a good idea if anybody takes issue with the interpretation chosen for xpx.expand_dims. But the milestone can probably be bumped (or removed until that happens).

  10. modified the milestones: v2024, v2025 on Jan 20, 2025
  11. 1 remaining item

  12. kgryte commented on Nov 13, 2025

    @kgryte
    Contributor

    @asmeurer Do you have any thoughts on Jake's proposal above?

  13. lucascolley commented on Nov 13, 2025

    @lucascolley
    Member
  14. kgryte commented on Nov 13, 2025

    @kgryte
    Contributor

    @lucascolley Thank you for that data point. I suppose the concern is ensuring Jake's proposed semantics match what is currently done in the ecosystem (apart from PyTorch). Do you have any insight here?

  15. lucascolley commented on Nov 13, 2025

    @lucascolley
    Member

    no particular insight, sorry

  16. asmeurer commented on Nov 13, 2025

    @asmeurer
    Member

    I think Jake's logic sounds reasonable. It seems like it should probably remove all the ambiguities, although you might want to write some hypothesis tests to verify that.

  17. asmeurer commented on Nov 13, 2025

    @asmeurer
    Member

    Also I'm not 100% sure how it works with what NumPy currently does. I expect if it doesn't match NumPy could be changed to match (i.e., NumPy might be giving errors in some cases where it isn't necessary), but we'd have to double check what it is doing.

  18. kgryte commented on Nov 13, 2025

    @kgryte
    Contributor

    Also I'm not 100% sure how it works with what NumPy currently does. I expect if it doesn't match NumPy could be changed to match (i.e., NumPy might be giving errors in some cases where it isn't necessary), but we'd have to double check what it is doing.

    @ev-br Is this something you'd be willing to investigate?

  19. added
    AcceptedRFC feature request which has been accepted.
    API changeChanges to existing functions or objects in the API.
    and removed on Feb 5, 2026
  20. ev-br commented on Feb 5, 2026

    @ev-br
    Member

    Here's a small hypothesis test: data-apis/array-api-tests#414 to verify the logic of #760 (comment), esp the last comment, "the behavior is semantically equivalent..."

    It seems to pass with 10_000 examples locally for numpy, jax and a torch wrapper of data-apis/array-api-compat#385 (which itself is lifted straight from numpy, https://github.com/numpy/numpy/blob/fabf1844667c81344ec0d0ab4958cde9cb9a845c/numpy/lib/_shape_base_impl.py#L596-L602)

    EDIT: data-apis/array-api-tests#414 with 100_000 examples passes locally with array_api_compat.{numpy,cupy,torch} and jax.numpy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    API changeChanges to existing functions or objects in the API.AcceptedRFC feature request which has been accepted.RFCRequest for comments. Feature requests and proposed changes.topic: ManipulationArray manipulation and transformation.

    Type

    No type

    Projects

    • Status
      Stage 0

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions