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

Handle interrupts better in the notebook #7702

Closed
boothby opened this issue Dec 16, 2009 · 8 comments
Closed

Handle interrupts better in the notebook #7702

boothby opened this issue Dec 16, 2009 · 8 comments

Comments

@boothby
Copy link

boothby commented Dec 16, 2009

Interrupting the notebook is less robust than it used to be. The attached worksheet is an example where the notebook fails to interrupt. When this happens, the notebook acts as though it's interrupted.

Component: notebook

Reviewer: Jeroen Demeyer

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

@boothby
Copy link
Author

boothby commented Dec 16, 2009

Attachment: interrupt test.sws.gz

@williamstein
Copy link
Contributor

comment:1

I have verified this. It is a very bad bug, since it means that one thinks the notebook got interrupted, but it didn't. This results in a seemingly totally broken worksheet, which must cause massive confusion to everybody, to say the least.

@williamstein
Copy link
Contributor

comment:2

(Oh, and it is surely my fault since I rewrote the interrupt code in the notebook separation.)

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 18, 2010

comment:3

The patch at #5712 should fix the problem.

@jdemeyer
Copy link

comment:4

Hard to say what's going on here. My guess would be a bare except: somewhere catching the KeyboardInterrupt.

@jdemeyer
Copy link

jdemeyer commented May 7, 2011

comment:5

I cannot reproduce the problem anymore in recent versions of Sage, so I assume it got fixed by #9678.

@jdemeyer
Copy link

jdemeyer commented May 8, 2011

Changed work issues from interrupt notebook to none

@jdemeyer
Copy link

jdemeyer commented May 8, 2011

Reviewer: Jeroen Demeyer

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

3 participants