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

chromatic_polynomial incorrectly blocks control-c #3125

Closed
williamstein opened this issue May 7, 2008 · 3 comments
Closed

chromatic_polynomial incorrectly blocks control-c #3125

williamstein opened this issue May 7, 2008 · 3 comments

Comments

@williamstein
Copy link
Contributor

Try this:

sage: graphs.CubeGraph(5).chromatic_polynomial()
[control-c]

control-c is ignored. Probably somebody doesn't understand _sig_on/_sig_off!

Component: combinatorics

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

@williamstein williamstein added this to the sage-3.0.2 milestone May 7, 2008
@rlmill rlmill mannequin assigned rlmill and unassigned mwhansen May 7, 2008
@rlmill
Copy link
Mannequin

rlmill mannequin commented May 7, 2008

comment:2

Attachment: trac3125-chrompoly_sig.patch.gz

After attached patch:

sage: graphs.CubeGraph(5).chromatic_polynomial()
^C---------------------------------------------------------------------------
<type 'exceptions.KeyboardInterrupt'>     Traceback (most recent call last)

/Users/rlmill/sage-3.0.1/<ipython console> in <module>()

/Users/rlmill/sage-3.0.1/local/lib/python/site-packages/sage/graphs/graph.py in chromatic_polynomial(self)
   7099         """
   7100         from sage.graphs.chrompoly import chromatic_polynomial
-> 7101         return chromatic_polynomial(self)
   7102 
   7103     def chromatic_number(self):

<type 'exceptions.KeyboardInterrupt'>: 

@rlmill rlmill mannequin added the s: needs review label May 10, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented May 11, 2008

comment:4

Patch is good.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented May 11, 2008

comment:5

Merged in Sage 3.0.2.alpha0

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

2 participants