[Numpy-discussion] Remove numpy/compat/_inspect.py ?
Charles R Harris
charlesr.harris at gmail.com
Fri Aug 1 09:03:38 EDT 2014
On Fri, Aug 1, 2014 at 6:37 AM, Robert Kern <robert.kern at gmail.com> wrote:
> On Fri, Aug 1, 2014 at 12:57 PM, Charles R Harris
> <charlesr.harris at gmail.com> wrote:
> > On Thu, Jul 31, 2014 at 11:46 PM, David Cournapeau <cournape at gmail.com>
> > wrote:
> >> The docstring at the beginning of the module is still relevant AFAIK: it
> >> was about decreasing import times. See
> >> On Fri, Aug 1, 2014 at 10:27 AM, Charles R Harris
> >> <charlesr.harris at gmail.com> wrote:
> >>> Hi All,
> >>> The _inspect.py function looks like a numpy version of the python
> >>> function. ISTR that is was a work around for problems with the early
> >>> versions, but that would have been back in 2009.
> >>> Thoughts?
> > It's only used in one function.
> Yes, one function that is called at startup, so no, a local import of
> the stdlib inspect module would not help.
> > def get_object_signature(obj):
> > """
> > Get the signature from obj
> > """
> > try:
> > sig = formatargspec(*getargspec(obj))
> > except TypeError as errmsg:
> > sig = ''
> > # msg = "Unable to retrieve the signature of %s '%s'\n"\
> > # "(Initial error message: %s)"
> > # warnings.warn(msg % (type(obj),
> > # getattr(obj, '__name__', '???'),
> > # errmsg))
> > return sig
> > Where a local import would do as well. It also has bugs, so evidently
> > called often ;)
> What bugs? Any bugs relevant to the objects that
> get_object_signature() is called with? It does not have to work for
> anything else but those.
Undefined variables in getargs. The only two functions used from the module
are very small and could simply be brought into `ma/core.py`. The python
inspect module is used elsewhere...
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the NumPy-Discussion