notebook: issue with indentation of first line being wrong

[williamstein]

On 11/27/2009 05:47 AM, Minh Nguyen wrote:
> On Fri, Nov 27, 2009 at 9:10 PM, Yotam Avital <> wrote:
>> for i in range (1,5):
>>     print '%6s %6s %6s'%(i, i^2, i^3)

I think *part* of the problem could be line 294 of sagenb.interfaces.expect:

       s = s.strip().rstrip(self._prompt)


Replacing this with

       s = s.rstrip(self._prompt)

appears to restore the expected spacing.  But quitting and reopening the
worksheet puts

1      1      1
    2      4      8
    3      9     27
    4     16     64

in the output cell.  I think the problem here is line 910 (or so) of
sagenb.notebook.cell:

           out = '///\n' + out.strip()


Replacing this with

           out = '///\n' + out.strip('\n')

seems to solve this problem.  It also makes the text representation of
the worksheet more compact.

Note: I haven't tested these changes extensively.

Component: notebook

Author: Mitesh Patel

Reviewer: Tim Dumol

Merged: sagenb-0.6

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

[qed777]

Attachment: trac_7648-missing_indent.patch.gz

Make the changes in the ticket description. sagenb repo.

[qed777]

Author: Mitesh Patel

[qed777]

comment:1

The Se tests pass. Unfortunately, sage -t sagenb/notebook yields several

A mysterious error (perhaps a memory error?) occurred, which may have crashed doctest.

Please see #7650.

[qed777]

comment:2

The patch for #7663 (and #7924) clashes with this one. Reconciling them should be easy.

[TimDumol]

comment:3

Good job.

[TimDumol]

Reviewer: Tim Dumol

[TimDumol]

Merged: sagenb-0.6