MAINT: don't install partial numpy.random C/Cython API. #14562
Conversation
See https://mail.python.org/pipermail/numpy-discussion/2019-September/080088.html for discussion. We need to do this right, and add tests and docs. All this PR does is not install bitgen.h, common.pxd and bit_generator.pxd (they're still shipped in the sdist, that is needed). Should be backported to 1.17.x
|
IMO we should remove the documentation from 1.17, but redesign it in master. I still don't have a good feel for what the correct C-API looks like or how we advise people consume it: whether it is the c file headers in |
Sounds good. I'll do that in the backport PR and will open a separate issue to make sure we don't forget about those docs for 1.18.0
Me neither. I just checked what Numba does for |
|
Still need to remove cython files on line 17 of the |
I don't think so. |
|
Thanks @rgommers. Hopefully we can redo this in a cleaner way soon |
See https://mail.python.org/pipermail/numpy-discussion/2019-September/080088.html
for discussion. We need to do this right, and add tests and docs.
All this PR does is not install
bitgen.h,common.pxdandbit_generator.pxd(they're still shipped in the sdist, that is needed).
Should be backported to 1.17.x
For https://numpy.org/devdocs/reference/random/extending.html#cython, we can either leave that in master and only remove it in 1.17.x, or remove it in master until we're ready to add the functionality - what is preferred?