Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DEP: remove untested, non-numeric code #12541

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

Closed
mattip opened this issue Dec 13, 2018 · 4 comments
Closed

DEP: remove untested, non-numeric code #12541

mattip opened this issue Dec 13, 2018 · 4 comments

Comments

@mattip
Copy link
Member

mattip commented Dec 13, 2018

In trying to make sense of our header files, I noticed we have a npy_interrupt.h file that defines a way to allow C code to respond to interrupts. The code is not tested. The only place it is used internally is in int_asbuffer, which interprets a python int as a pointer to a block of memory, tries to determine the size of the block, and returns a buffer object. It also is not tested.

At some point we should remove these since they are not really part of NumPy.

@mattip
Copy link
Member Author

mattip commented Sep 23, 2019

This would require deprecating _PyArray_SigintHandler, which is a public API function

@Kai-Striega
Copy link
Member

@mattip I've got some time to contribute at the moment. I'd need some guidance on how to get started (I don't have problems building/testing NumPy, only context on this specific issue)

@mattip
Copy link
Member Author

mattip commented Sep 29, 2022

You can look at other API function deprecations, like in #21188 here. Note this needs a release note, see the readme

seberg added a commit to seberg/numpy that referenced this issue Nov 21, 2022
We do not use these in NumPy anymore, and at this point the whole
`npy_interrupt.h` header only exists in case someone is using it
out there.
We may wish to just remove it at some point, vendoring the header is
simple enough after all (and no known downstream usage exists).

See also numpygh-7545, numpygh-12541
@seberg
Copy link
Member

seberg commented Jun 23, 2023

This is gone.

@seberg seberg closed this as completed Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants