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

DEP: Disable PyUFunc_GenericFunction and PyUFunc_SetUsesArraysAsData #18697

Merged
merged 2 commits into from Mar 30, 2021

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).
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.

None yet

2 participants