Opened 13 years ago

Closed 13 years ago

#5020 closed defect (fixed)

[with patch, positive review] auto-cells do not automaticall evaluate (or at least update)

Reported by: mhansen Owned by: boothby
Priority: major Milestone: sage-3.3
Component: notebook Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Notebook cells that start with "#auto" do not automatically evaluate on the when a worksheet is opened.

Attachments (1)

trac_5020.patch (792 bytes) - added by mhansen 13 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 13 years ago by mhampton

They do on sagenb, which is running sage-3.2.3, but do not seem to on my laptop (also running sage-3.2.3)...?

comment:2 Changed 13 years ago by mhampton

As a clarification, #auto seems totally broken for me on an unpatched sage-3.2.3 (on intel OS X 10.5). That install does have a lot of optional spkgs and the tinymce spkgs, but Dan Drake reports it working with the tinymce patches+spkgs, so I don't know what's going on.

Changed 13 years ago by mhansen

comment:3 Changed 13 years ago by mhansen

  • Summary changed from auto-cells do not automaticall evaluate (or at least update) to [with patch, needs review] auto-cells do not automaticall evaluate (or at least update)

I've add a test to the notebook selenium test suite which tests this.

comment:4 Changed 13 years ago by mhansen

I should mention why this fixes things :-)

The cells are evaluated when .sage() is called, but the HTML send to the web browser was generated before that.

comment:5 Changed 13 years ago by ddrake

  • Summary changed from [with patch, needs review] auto-cells do not automaticall evaluate (or at least update) to [with patch, positive review] auto-cells do not automaticall evaluate (or at least update)

Looks good. Positive review.

comment:6 Changed 13 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 3.3.alpha0

Note: See TracTickets for help on using tickets.