notebook: default values for variables are printed incorrectly in docstrings

[jhpalmieri]

From sage-notebook:

> I notice that in Sage 4.2, the version of sageinspect in the notebook 
> doesn't match the one in sage.misc -- the one in sagenb/misc is 
> missing the fix from Trac #6848.  As a result, 
> {{{ 
> RDF.random_element? 
> }}} 
> produces incorrect output, as noted on the ticket.  Should this be 
> fixed, or was the fix intentionally omitted because it uses "eval"? 

For the fix, see the patch at #6848, especially the new lines 269-270 (and the associated doctest fixes).

CC: @williamstein

Component: notebook

Author: John Palmieri

Reviewer: Mitesh Patel, Tim Dumol

Issue created by migration from https://trac.sagemath.org/ticket/7316

[qed777]

Attachment: trac_7316-sageinspect_defn.patch.gz

Fix Cython docstring "Definition". Rebase of sageinspect part of #6848. Apply to sagenb repository.

[qed777]

Author: John Palmieri

[qed777]

comment:1

All doctests pass, if I copy the patched sagenb.misc.sageinspect.py to $SAGE_ROOT/devel/sage/sage, say, and run sage -t sageinspect.py in that directory.

To the extent that it counts, my review is positive.

[jhpalmieri]

comment:2

Is this now a duplicate of #7349?

[TimDumol]

comment:3

Doctests count and the the bugs are fixed. Positive review.

[TimDumol]

Reviewer: Mitesh Patel, Tim Dumol

[williamstein]

comment:4

merged into sagenb-0.4.2 (sage-4.2.1)