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

avoid subtle interaction between importing multiprocessing and twisted #8785

Closed
williamstein opened this issue Apr 27, 2010 · 8 comments
Closed
Assignees
Milestone

Comments

@williamstein
Copy link
Contributor

It turns out that on some platforms, importing multiprocessing, then twisted, leads to an "int object is not callable" TypeError. This breaks devel/sage/sage/all.py's quit_sage function, causing a big traceback at exit. This could also cause great confusion for people writing a program that uses @parallel('multiprocessing') followed by anything involving twisted.

A simple fix is to import the relevant part of twisted before using multiprocessing in @parallel. The attached patch does this.

NOTE: The system that exhibits this is in a corporate setting, and no devs have systems where this can be replicated at present, unfortunately. So please do NOT revert this just because you don't see the problem on your laptop!

Component: misc

Author: William Stein

Reviewer: Mariah Lennox

Merged: sage-4.4.1.alpha0

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

@williamstein

This comment has been minimized.

@williamstein
Copy link
Contributor Author

comment:2

Attachment: trac_8785.patch.gz

@sagetrac-mariah
Copy link
Mannequin

sagetrac-mariah mannequin commented Apr 27, 2010

comment:4

I tried it and it works!

@williamstein
Copy link
Contributor Author

Merged: 4.4.1.alpha0

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Apr 29, 2010

Reviewer: Mariah Lennox

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Apr 29, 2010

Author: William Stein

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 3, 2010

Changed merged from 4.4.1.alpha0 to sage-4.4.1.alpha0

@videlec
Copy link
Contributor

videlec commented Jan 10, 2016

comment:8

Would be more informative to write explicitely on which hardware/OS it failed. "corporate settings" is more than vague. Was there any upstream report? This problem might have been solved since then!

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