Ticket #3075 (assigned enhancement)

Opened 7 months ago

Last modified 3 months ago

notebook -- add a "test" option

Reported by: was Assigned to: mhansen (accepted)
Priority: major Milestone: sage-3.2.2
Component: notebook Keywords:
Cc:

Description

Click Action ---> Test and all cells will be evaluated and their output compared with the output last time. Any time the output differs both outputs are shown with the bad one in RED.

Or something like that...

Change History

05/01/2008 03:28:36 PM changed by was

  • type changed from defect to enhancement.

05/02/2008 10:26:24 AM changed by was 

10:13 < mabshoff> And I have been slipping in FreeBSD fixes when no one was looking ;)
10:16 < jason|log> okay, "flaky" == in one cell, I have a=[] and i=1, the next I have a.append(i).  
                   Evaluate all sometimes gives me an error in the second cell and says that 'a' is not 
                   defined.
10:16 < jason|log> I can't reproduce it on sage.math in the few minutes I tried.
10:18 < jason|log> wstein: could the problem be the asynchronous requests to evaluate the cells?  One 
                   request gets ahead of another sometimes?
10:19 < jason|log> (not in the sending, but in the server receiving the requests?)
10:19 < wstein> Yes, that is possible.
10:20 < wstein> The right fix would be to make a single "evaluateall" command that gets sent
10:20 < wstein> to the notebook server.
10:20 < wstein> Right now a stupid evaluate_all javascript function calls evaluate_cell on each cell.

09/07/2008 06:29:08 PM changed by mhansen

  • owner changed from boothby to mhansen.
  • status changed from new to assigned.

09/07/2008 06:33:34 PM changed by mhansen

This patch fixes the problems with evaluate all.

09/07/2008 06:33:44 PM changed by mhansen

  • summary changed from notebook -- add a "test" option to [with patch, needs review] notebook -- add a "test" option.

09/08/2008 04:23:04 AM changed by TimothyClemans

The patch by mhansen doesn't add a "test" option, so I think it should be on a different ticket.

09/08/2008 06:54:32 AM changed by mhansen

  • summary changed from [with patch, needs review] notebook -- add a "test" option to notebook -- add a "test" option.

I made #4078.

Do we even really want this feature for the notebook? I'd vote for marking this ticket as invalid.

09/08/2008 06:58:56 AM changed by TimothyClemans

I don't want it.