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.

Sign up for GitHub

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

Jump to bottom

polymake deadlocks from popen3 call #403

Closed
sagetrac-mhampton mannequin opened this issue Jul 12, 2007 · 3 comments
Closed

polymake deadlocks from popen3 call #403

sagetrac-mhampton mannequin opened this issue Jul 12, 2007 · 3 comments
Assignees
@williamstein
Labels
c: interfaces p: minor / 4 t: bug
Milestone
sage-2.8.3

Comments

@sagetrac-mhampton
Copy link
Mannequin

sagetrac-mhampton mannequin commented Jul 12, 2007

When doing large enough calculations, popen3 deadlocks in the cmd method of the Polytope class. The attached patch fixes this by using the subprocess module instead.

Component: interfaces

Keywords: polymake, polytope

Issue created by migration from https://trac.sagemath.org/ticket/403
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Aug 20, 2007

comment:1

Attachment: polytope.patch.gz

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-2.8.3 milestone Aug 20, 2007
@williamstein
Copy link
Contributor

comment:2

I changed the priority to minor because polymake is an optional package.

@williamstein
Copy link
Contributor

comment:3

This is applied now for sage-2.8.3.

tobiasdiez pushed a commit to tobiasdiez/sage that referenced this issue Feb 22, 2024
@marcelotrevisani
Remove pre-commit github actions and use pre-commit.ci instead (sagem…
c9e1230 
…ath#403)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@williamstein williamstein

Labels
c: interfaces p: minor / 4 t: bug
Projects
None yet
Milestone
sage-2.8.3
Development

No branches or pull requests
1 participant
@williamstein