Skip to content
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

MAINT: Replace NumPy aliased functions #9888

Merged
merged 2 commits into from
Mar 1, 2019

Conversation

pentschev
Copy link
Contributor

In numpy/numpy#10653, NumPy has replaced duplicate
implementations by aliased functions. Such aliases may not be available in other
NumPy-like libraries (e.g., CuPy), thus breaking compatibility after the
introduction of array_function, as described in NEP-18 [1].

[1] https://www.numpy.org/neps/nep-0018-array-function-protocol.html

cc @mrocklin

In numpy/numpy#10653, NumPy has replaced duplicate
implementations by aliased functions. Such aliases may not be available in other
NumPy-like libraries (e.g., CuPy), thus breaking compatibility after the
introduction of __array_function__, as described in NEP-18 [1].

[1] https://www.numpy.org/neps/nep-0018-array-function-protocol.html
Copy link
Member

@perimosocordiae perimosocordiae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me. More consistency across the codebase is always nice.

@rgommers rgommers added the maintenance Items related to regular maintenance tasks label Mar 1, 2019
@rgommers rgommers merged commit 54e1727 into scipy:master Mar 1, 2019
@rgommers
Copy link
Member

rgommers commented Mar 1, 2019

LGTM too, thanks @pentschev

@rgommers rgommers added this to the 1.3.0 milestone Mar 1, 2019
@pentschev
Copy link
Contributor Author

Thanks @perimosocordiae and @rgommers for taking the time to review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Items related to regular maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants