-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
DEP: remove deprecated (and private) numpy.testing submodules. #14567
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
Conversation
Removes these files that were deprecated since 1.15.0: - numpy/testing/decorators.py - numpy/testing/noseclasses.py - numpy/testing/nosetester.py This also resolves a failure in the recently introduced tests in `test_public_api.py` (see numpygh-14454). Closes numpygh-14566
4d30014
to
d8b1a87
Compare
There are still folks using the NumPy nose framework so it may be a bit early to delete the files. Most of the linux distributions have patched versions of nose, so that still works. If we do remove the files there needs to be a release note. |
Nothing is gone, the correct imports are and have always been
These files were always private, they just missed an underscore. |
@charris can we ping those projects to encourage them to deprecate the old imports? |
OK, looks like there are workarounds for any that need it. I'll do a wheels build after this is merged to check. Thanks Ralf. |
OK, the travis builds are working, but the appveyor builds are failing. I don't know if that is related. Note that a new version of setuptools was released a month ago. I suspect that maybe things were fixed to run that check, as the numpy version cannot be available before the build takes place. |
That Appveyor failure does not look related. It does look very odd though.
Indeed, that can't be right, looks like a regression introduced in |
Or it got triggered by @mattip's |
I'm suspecting the latter, the code in setuptools is three years old. |
Removes these files that were deprecated since 1.15.0:
This also resolves a failure in the recently introduced tests in
test_public_api.py
(see gh-14454). Closes gh-14566