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

@parallel -- dramatically improve by rewriting with fork directly, using files, timeouts, controlling interfaces, etc. #6967

Closed
williamstein opened this issue Sep 20, 2009 · 12 comments
Closed

@parallel -- dramatically improve by rewriting with fork directly, using files, timeouts, controlling interfaces, etc. #6967

williamstein opened this issue Sep 20, 2009 · 12 comments
Assignees
@sagetrac-cwitty
Labels
p: major / 3 t: bug
Milestone
sage-4.3.1

Comments

@williamstein
Copy link
Contributor

TODO:

Component: misc

Author: William Stein

Reviewer: Robert Miller

Merged: sage-4.3.1.rc1

Issue created by migration from https://trac.sagemath.org/ticket/6967
@williamstein williamstein added this to the sage-4.3.1 milestone Sep 20, 2009
@williamstein
Copy link
Contributor Author

first usable version

@williamstein
Copy link
Contributor Author

comment:1

Attachment: trac_6967-part1.patch.gz

@williamstein

This comment has been minimized.

Sign in to view
@qed777
Copy link
Mannequin

qed777 mannequin commented Nov 18, 2009

comment:2

Should we also allow each child process to pull off a large chunk of the input (e.g., from a deque), when it's more efficient? Determine the chunk size dynamically, a la timeit?

@williamstein
Copy link
Contributor Author

comment:3

This fixes major bugs in @parallel sucking. Without this, @parallel gets confused by Sage-isms, preparsing, state, etc., and really hasn't taken off as a result. This fixes all that.

@williamstein
Copy link
Contributor Author

comment:4

This also fixes a very serious bug in sage.interfaces.quit which will lead to zombie processes being left around, and improves doctest coverage.

@williamstein
Copy link
Contributor Author

Attachment: trac_6967-part2.patch.gz

@williamstein
Copy link
Contributor Author

Attachment: trac_6967-part3.patch.gz

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

comment:5

Awesome!!!

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Author: William Stein

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Reviewer: Robert Miller

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Merged: sage-4.3.1.rc1

@rlmill rlmill mannequin removed the s: positive review label Jan 19, 2010
@rlmill rlmill mannequin closed this as completed Jan 19, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-cwitty sagetrac-cwitty

Labels
p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.3.1
Development

No branches or pull requests
1 participant
@williamstein