The argspecs of extension function/methods is broken in the Sphinx documentation

[hivert]

In the current Sphinx HTML doc, the extenstion function and methods have no arguments setup.

See for example the documentation of

reference/sage/symbolic/expression.html#sage.symbolic.expression.Expression.N

which was ok (see there) in Sage 4.8. The problem was introduced in sage-5.0.beta8.

We should add a regression test on that kinds of problem.

Apply:

• 12849_doctest.patch
• trac_12849-extfunc_argspec_html_fix-fh.patch

[jdemeyer]

Doctest for the issue

[jdemeyer]

I created a doctest for this issue, which obviously fails on recent Sage betas.

[hivert]

Replying to jdemeyer:

I created a doctest for this issue, which obviously fails on recent Sage betas.

Thanks ! I tried to figureout a way to call directly Sphinx but this is much easier. I'll try to work on this this afternoon. Do you have somewhere the various beta compiled so that I can rsync them on boxen to help bissecting ? So far the few guesses I made to find the culprit were wrong.

[jdemeyer]

sage-5.0.beta5 is still okay.

[jdemeyer]

Replying to hivert:

Replying to jdemeyer:

I created a doctest for this issue, which obviously fails on recent Sage betas.

Thanks ! I tried to figureout a way to call directly Sphinx but this is much easier. I'll try to work on this this afternoon. Do you have somewhere the various beta compiled so that I can rsync them on boxen to help bissecting ? So far the few guesses I made to find the culprit were wrong.

Look at  http://boxen.math.washington.edu/home/release/. There should be binaries for all Sage betas, made on sage.math or boxen.math.

[hivert]

It seems that I have a fix, but I don't understand how it worked before ! Still looking.

Florent

[jdemeyer]

It looks like this was introduced in sage-5.0.beta8.

[jdemeyer]

The culprit is #9128.

[hivert]

Thanks for investigating.

Replying to jdemeyer:

The culprit is #9128.

Strange ! It was an obvious candidate and I'm pretty sure I started by checking this one. Still I don't understand what could cause that in #9128. I'm testing my fix.

Florent

[hivert]

Hi,

I got the fix ! I'm definitely the culprit. For strange reason the following lines were removed by #9128. Putting them back should fix the problem.

doc/common/conf.py

@@ -576 +576 @@
     newnode.append(contnode)
     return newnode
 
+from sage.misc.sageinspect import sage_getargspec
+autodoc_builtin_argspec = sage_getargspec
 
 def setup(app):
     app.connect('autodoc-process-docstring', process_docstring_cython)

[hivert]

Jeroen: I'm reviewing your patch. Can you review mine ?

Florent

[jdemeyer]

Replying to hivert:

Can you review mine ?

Sorry, no. I'm not at all familiar with the docbuilding process.

[mhansen]

Florent's patch is good, as is Jeroen (even if it is a bit sensitive to the particular HTML output used by Sphinx).

[hivert]

Replying to mhansen:

Florent's patch is good, as is Jeroen (even if it is a bit sensitive to the particular HTML output used by Sphinx).

Thanks Mike !

[hthomas]

Hi!

The doctest introduced by this patch, at line 22 of sage/misc/sagedoc.py, is causing the sagepad.org patchbot to fail on some (trivial, unrelated) patches. The problem seems to be that the doctest is trying to check the documentation when the documentation isn't built.

I don't know if this is really a problem with this patch, or with the patchbot, but I thought I would try reporting it here. If I should try elsewhere, please let me know!

You can see this failure at #12943, which is a completely trivial patch. (It's also making the sagepad.org patchbot fail on #10527. If you look at that ticket, though, don't look at the log for Volker's patchbot, which is having a different problem.)

cheers,

Hugh

[jhpalmieri]

A complete set of documentation is an integral part of a Sage build, so doctests are supposed to fail if the documentation hasn't been built. This is not because of this ticket, I don't think.

[hthomas]

Thanks. The issue seems to be with the particular patchbot -- I have posted to sage-devel about it.