Opened 12 years ago

Closed 12 years ago

#7663 closed defect (fixed)

notebook -- synchronization code suppresses printing of certain characters

Reported by: was Owned by: was
Priority: major Milestone: sage-4.3.1
Component: notebook Keywords:
Cc: Merged in: sagenb-0.6
Authors: Mitesh Patel Reviewers: Tim Dumol
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

This is weird. Occassionaly, in the SAGE notebook, version 4.2.1, the
last characters of output after evaluating a cell are supressed. I've
tried all of string.printable: the only characters supressed are "S",
"A", "G", "E" and "_". No kidding.
 Just try:

print "ASAVFDBAAGGG___EEESS"
///
ASAVFDB


print "ASAVFDBAAGGG___EEESS."
///
ASAVFDBAAGGG___EEESS.

 Is this a private joke or an amazing coincidence?

 Regards
Pablo

Attachments (3)

trac_7663-rstrip_prompt.patch (752 bytes) - added by mpatel 12 years ago.
Don't use rstrip on internal prompt. sagenb repo.
trac_7663-rstrip_prompt.2.patch (762 bytes) - added by timdumol 12 years ago.
Rebased on #7648.
trac_7663-rstrip_prompt.3.patch (723 bytes) - added by mpatel 12 years ago.
Remove extra line to fix #7648. Replaces previous.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 12 years ago by mpatel

For what it's worth, I don't see this problem with the latest at #6855. I may have inadvertently fixed it.

comment:2 Changed 12 years ago by kcrisman

  • Summary changed from notebook -- synchronization code surprises printing of certain characters to notebook -- synchronization code suppresses printing of certain characters

comment:3 Changed 12 years ago by mpatel

Possibly related: #7410. #7666 may fix both.

comment:4 Changed 12 years ago by mpatel

Actually, see #7924. In particular,

sage: from sagenb.interfaces.expect import WorksheetProcess_ExpectImplementation
sage: wp = WorksheetProcess_ExpectImplementation() 
sage: wp.execute('print "ASAVFDBAAGGG___EEESS"')
sage: wp.output_status()
Output Status:
        output: 'ASAVFDB'
        filenames: []
        done: True
sage: wp.execute('print "ASAVFDBAAGGG___EEESS."')
sage: wp.output_status()
Output Status:
        output: 'ASAVFDBAAGGG___EEESS.'
        filenames: []
        done: True

Changed 12 years ago by mpatel

Don't use rstrip on internal prompt. sagenb repo.

comment:5 Changed 12 years ago by mpatel

  • Authors set to Mitesh Patel
  • Status changed from new to needs_review

comment:6 Changed 12 years ago by mpatel

The patch here clashes with that at #7648, but it should be easy to reconcile them.

Changed 12 years ago by timdumol

Rebased on #7648.

comment:7 Changed 12 years ago by timdumol

  • Reviewers set to Tim Dumol
  • Status changed from needs_review to positive_review

LGTM.

Changed 12 years ago by mpatel

Remove extra line to fix #7648. Replaces previous.

comment:8 Changed 12 years ago by mpatel

V3 drops s = s.strip(), to maintain #7648.

comment:9 Changed 12 years ago by timdumol

  • Merged in set to sagenb-0.6
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.