Skip to content

Conversation

@seberg
Copy link
Member

@seberg seberg commented Mar 29, 2021

Both functions have been deprecated in 1.19.x without complaints
(so far). The first function is largely identical to a Python ufunc
call. The second function had served internal NumPy purposes until
NumPy 1.6 (probably), and since then had a comment that it should
probably be removed (it was not documented).

Both functions have been deprecated in 1.19.x without complaints
(so far).  The first function is largely identical to a Python ufunc
call.  The second function had served internal NumPy purposes until
NumPy 1.6 (probably), and since then had a comment that it should
probably be removed (it was not documented).
@seberg seberg force-pushed the finish-deprecations-umath branch from 6d12ecd to a115369 Compare March 29, 2021 19:03
Copy link
Member

@mattip mattip left a comment

Choose a reason for hiding this comment

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

LGTM. It seems there are no tests for this function and its deprecation.

@seberg
Copy link
Member Author

seberg commented Mar 30, 2021

🤷 I can add a small test that it raises if you like. These functions where never tested, and unused for probably around a decade...

@mattip mattip merged commit fb60eb7 into numpy:main Mar 30, 2021
@mattip
Copy link
Member

mattip commented Mar 30, 2021

Thanks @seberg

@seberg seberg deleted the finish-deprecations-umath branch March 30, 2021 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants