numexpr icon indicating copy to clipboard operation
numexpr copied to clipboard

MNT: use compat functions for accessing dtype state

Open tacaswell opened this issue 1 year ago • 1 comments

This complies with the numpy main branch, but did not test with older versions of numpy (I suspect it wont work due to missing functions)

The np_2_compat.h is designed to be vendored down stream, I think the question is if numexpr wants the whole thing or just the minimal sub-set needed to fix these problems.

tacaswell avatar Mar 08 '24 22:03 tacaswell

Ok, your PR does not compile yet. I think there should be no problem in vendoring np_2_compat.h, but at any rate, you could start by adding it here and decide later. Thanks!

FrancescAlted avatar Mar 09 '24 04:03 FrancescAlted

Overridden by https://github.com/pydata/numexpr/pull/478

FrancescAlted avatar Apr 01 '24 15:04 FrancescAlted

🎉 Thank you! I did not have time to come back and finish this and it looks like @neutrinoceros did a better job than I would have anyway!

tacaswell avatar Apr 01 '24 21:04 tacaswell

Oh my, I absolutely didn't realize I was scoping your PR, sorry I didn't look hard enough !

neutrinoceros avatar Apr 01 '24 22:04 neutrinoceros