Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not automatically evaluate interact cells in notebook #4363

Closed
kcrisman opened this issue Oct 24, 2008 · 8 comments
Closed

Do not automatically evaluate interact cells in notebook #4363

kcrisman opened this issue Oct 24, 2008 · 8 comments

Comments

@kcrisman
Copy link
Member

Current behavior is that @interact cells automatically evaluate upon opening a worksheet. This can cause problems if (for instance) the cell depends on other cells which are not automatically evaluated, and also can take a long time if there are lots of them.

Since other cells do not auto-evaluate, and since this functionality still is easily available by putting #auto in the cell, this ticket calls for the current behavior to be changed.

Component: notebook

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

@jasongrout
Copy link
Member

comment:1

I second that this should be changed.

@mwhansen
Copy link
Contributor

comment:2

I've also added this as a test case to the notebook test suite.

@williamstein
Copy link
Contributor

comment:3

I think to solve this trac ticket, it is necessary that:

  1. Output of interact cells should be completely empty when a worksheet is open.

  2. @interact needs to work with #auto (or %auto?) because I have a project with a student that involves making a bunch of interacts that are all %hideall'd, so one sees only the controls.

@williamstein
Copy link
Contributor

comment:4

Just to emphasize, the patch above mysteriously doesn't work with #auto...

@mwhansen mwhansen changed the title Do not automatically evaluate interact cells in notebook [needs woork] Do not automatically evaluate interact cells in notebook Nov 14, 2008
@mwhansen mwhansen changed the title [needs woork] Do not automatically evaluate interact cells in notebook Do not automatically evaluate interact cells in notebook Dec 2, 2008
@mwhansen
Copy link
Contributor

Attachment: trac_4363.patch.gz

@mwhansen
Copy link
Contributor

comment:7

I've posted a patch which does not enqueue interact cells at startup and deletes their output so that users don't think that they're running.

I've also made a ticket to fix the auto-evaluation at #5020 . I think this is a separate issue, and I'm looking into it.

@sagetrac-mhampton
Copy link
Mannequin

sagetrac-mhampton mannequin commented Jan 19, 2009

comment:8

This seems to solve the problem, and the auto-launch problem of #4947. My setup seems to have some issues with #auto cells, so I can't test the compatibility of this patch with that, but that is addressed in other ways by #5020.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 19, 2009

comment:9

Merged in Sage 3.3.alpha0

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.4.1, sage-3.3 Jan 19, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Jan 19, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants