Skip to content

Conversation

@eric-wieser
Copy link
Member

This function has been deprecated since fcee1ad (Numpy v1.0b3).

14 years is more than enough time for users to switch from it.

@eric-wieser eric-wieser added 07 - Deprecation 62 - Python API Changes or additions to the Python API. Mailing list should usually be notified. labels Aug 20, 2020
@eric-wieser eric-wieser force-pushed the expire-ctypes_load_library branch from dcd2986 to d852ecb Compare August 20, 2020 12:10
This function has been deprecated since fcee1ad (Numpy v1.0b3).

14 years is more than enough time for users to switch from it.
@eric-wieser eric-wieser force-pushed the expire-ctypes_load_library branch from d852ecb to caa70ea Compare August 20, 2020 12:12
@eric-wieser eric-wieser marked this pull request as ready for review August 20, 2020 12:12
@mattip
Copy link
Member

mattip commented Aug 20, 2020

This was missing from the deprecation tracking issue gh-11521, maybe because the deprecation was done via wrapping with numpy.lib.utils.deprecate. As far as I can tell this was the only use of deprecate (except for tests that test the functionality of it).

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

+1 from me

@seberg
Copy link
Member

seberg commented Aug 20, 2020

Hah, nice, thanks Eric.

@@ -0,0 +1,2 @@
* The 14-year deprecation of ``np.ctypeslib.ctypes_load_library`` is expired.
Use :func:`~numpy.ctypeslib.load_library` instead, which is identical.
Copy link
Member Author

Choose a reason for hiding this comment

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

Note that we seem to have two different styles of .expired notes - bullets and headers. I'm inclined to leave unifying them to when we release.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, we can see that later, a mix may even be OK as well (I am not sure how long the bullet ones are). I was just looking if I ignore Travis here, and I think I will, its bogged down for ages.

@seberg seberg merged commit 2810bec into numpy:master Aug 20, 2020
@eric-wieser eric-wieser deleted the expire-ctypes_load_library branch August 20, 2020 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

07 - Deprecation 62 - Python API Changes or additions to the Python API. Mailing list should usually be notified.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants