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

Speed regression for n_jobs=1 #483

Closed
lesteve opened this issue Jan 24, 2017 · 1 comment
Closed

Speed regression for n_jobs=1 #483

lesteve opened this issue Jan 24, 2017 · 1 comment
Assignees

Comments

@lesteve
Copy link
Member

@lesteve lesteve commented Jan 24, 2017

from scikit-learn/scikit-learn#8186 (comment):

Snippet reproducing the issue:

from joblib import Parallel, delayed

def func():
    return 42

inputs = range(1000)

%timeit Parallel(n_jobs=1)(delayed(func)() for _ in inputs)

The speed regression is way worse on Python 2:

Timings:

Python 2.7 Python 3.5
joblib 0.9.4 32.8 ms 38.6 ms
joblib master 667 ms 76.6 ms
@lesteve

This comment has been minimized.

Copy link
Member Author

@lesteve lesteve commented Jan 24, 2017

Profiling, the culprit is the timeout support and in particular this line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.