Repository navigation
Replace calls to _report_generic_error with a more useful error message. #15316
Description
Right now, many failures of np.dtype(obj) go to this path:
numpy/numpy/core/src/multiarray/descriptor.c
Lines 42 to 49 in 80fc0f5
For example
numpy/numpy/core/src/multiarray/descriptor.c
Lines 443 to 446 in 80fc0f5
We should replace some or all of these calls in this file with more helpful error messages, such as in the above case:
-_report_generic_error();
+PyErr_Format(PyExc_TypeError, "Field elements must be 2-tuples, got %R", item);If we replace all of them, then we should remove the _report_generic_error function entirely
Activity
faizan2700 commentedon Jan 16, 2020
Hello I want to contribute to this issue but I am having one problem please help.
After forking repo I have cloned it on my laptop and now from numpy directory I am trying to import numpy but it is giving error that I can't import numpy from there and exit from there and then import it. if I am in some other directory and the numpy directory path to sys.path and then import numpy it gives the same error. but if I have made some changes in source code and want to test the effect of it in interpreter or test numpy how do I do that when I can not import it?
My question is when you make changes in numpy how do you import the source of numpy and check that ?
eric-wieser commentedon Jan 16, 2020
I (on windows) normally uses
py -3.8 runtests.py -i, which opens an ipython shell with your in-development numpyhasansajedi commentedon Jan 18, 2020
When this error occured?
Can you leave here an input example?
ericmariasis commentedon Feb 1, 2020
I created a pull request for the enhancement.