BUG: Defer to autodoc for signatures

numpydoc/docscrape.py

@@ -566,22 +566,6 @@ def __init__(self, func, role='func', doc=None, config={}):
             doc = inspect.getdoc(func) or ''
         NumpyDocString.__init__(self, doc, config)
 
-        if not self['Signature'] and func is not None:
-            func, func_name = self.get_func()
-            try:
-                try:
-                    signature = str(inspect.signature(func))
-                except (AttributeError, ValueError):
-                    # try to read signature, backward compat for older Python
-                    if sys.version_info[0] >= 3:
-                        argspec = inspect.getfullargspec(func)
-                    else:
-                        argspec = inspect.getargspec(func)
-                    signature = inspect.formatargspec(*argspec)
-                signature = '%s%s' % (func_name, signature)
-            except TypeError:
-                signature = '%s()' % func_name
-            self['Signature'] = signature
 
     def get_func(self):
         func_name = getattr(self._f, '__name__', self.__class__.__name__)

numpydoc/numpydoc.py

@@ -203,12 +203,30 @@ def mangle_signature(app, what, name, obj, options, sig, retann):
     if not hasattr(obj, '__doc__'):
         return
     doc = get_doc_object(obj, config={'show_class_members': False})
-    sig = doc['Signature'] or getattr(obj, '__text_signature__', None)
+    sig = (doc['Signature'] or
+           _clean_text_signature(getattr(obj, '__text_signature__', None)))
     if sig:
         sig = re.sub("^[^(]*", "", sig)
         return sig, ''
 
 
+def _clean_text_signature(sig):
+    if sig is None:
+        return None
+    sig = re.sub(r"^[^(]*\(", "", sig)
+    sig = re.sub(r"\)", "", sig)
+    params = sig.replace(' ', '').split(',')
+    if '$self' in params:
+        params.remove('$self')
+    if '$module' in params:
+        params.remove('$module')
+    if '$type' in params:
+        params.remove('$type')
+    if '/' in params:
+        params.remove('/')
+    return '(' + ', '.join(params) + ')'
+
+
 def setup(app, get_doc_object_=get_doc_object):
     if not hasattr(app, 'add_config_value'):
         return  # probably called by nose, better bail out

numpydoc/tests/test_docscrape.py

@@ -699,7 +699,7 @@ def my_func(a, b, **kwargs):
         pass
 
     fdoc = FunctionDoc(func=my_func)
-    assert fdoc['Signature'] == 'my_func(a, b, **kwargs)'
+    assert fdoc['Signature'] == ''
 
 
 doc4 = NumpyDocString(

numpydoc/tests/test_full.py

@@ -53,12 +53,13 @@ def test_MyClass(sphinx_app):
         html = fid.read()
     # ensure that no autodoc weirdness ($) occurs
     assert '$self' not in html
+    assert '/,' not in html
     assert '__init__' in html  # inherited
     # escaped * chars should no longer be preceded by \'s,
     # if we see a \* in the output we know it's incorrect:
     assert r'\*' not in html
     # "self" should not be in the parameter list for the class:
-    assert 'self,' in html   # XXX should be "not in", bug!
+    assert 'self,' not in html
     # check xref was embedded properly (dict should link using xref):
     assert 'stdtypes.html#dict' in html
 

numpydoc/tests/test_numpydoc.py

@@ -1,6 +1,6 @@
 # -*- encoding:utf-8 -*-
 from copy import deepcopy
-from numpydoc.numpydoc import mangle_docstrings
+from numpydoc.numpydoc import mangle_docstrings, _clean_text_signature
 from numpydoc.xref import DEFAULT_LINKS
 from sphinx.ext.autodoc import ALL
 
@@ -64,6 +64,20 @@ def test_mangle_docstrings():
     assert 'upper' not in [x.strip() for x in lines]
 
 
+def test_clean_text_signature():
+    assert _clean_text_signature(None) is None
+    assert _clean_text_signature('func($self)') == '()'
+    assert (_clean_text_signature('func($self, *args, **kwargs)') ==
+            '(*args, **kwargs)')
+    assert _clean_text_signature('($self)') == '()'
+    assert _clean_text_signature('()') == '()'
+    assert _clean_text_signature('func()') == '()'
+    assert (_clean_text_signature('func($self, /, *args, **kwargs)') ==
+            '(*args, **kwargs)')
+    assert _clean_text_signature('($module)') == '()'
+    assert _clean_text_signature('func($type)') == '()'
+
+
 if __name__ == "__main__":
     import pytest
     pytest.main()