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

[MRG] Allow backend instance in Parallel constructor #449

Merged
merged 3 commits into from
Dec 19, 2016
Merged

[MRG] Allow backend instance in Parallel constructor #449

merged 3 commits into from
Dec 19, 2016

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented Dec 9, 2016 •
edited

Fix #448.

Still need to:

  • add some tests.

@lesteve lesteve force-pushed the allow-backend-instance-in-parallel-constructor branch 2 times, most recently from 813e8b0 to e9d98ac Compare December 14, 2016 22:45
@lesteve lesteve changed the title [WIP] Allow backend instance in Parallel constructor [MRG] Allow backend instance in Parallel constructor Dec 15, 2016
@lesteve
Copy link
Member Author

lesteve commented Dec 15, 2016

OK I have added some tests. @aabadie maybe you can have a quick look at this one before I merge?

I chatted with @ogrisel IRL and he was fine with the change.

@lesteve lesteve force-pushed the allow-backend-instance-in-parallel-constructor branch from e9d98ac to cbab672 Compare December 15, 2016 15:42
@lesteve
Copy link
Member Author

lesteve commented Dec 19, 2016

Merging this one.

@lesteve lesteve merged commit 1460b0b into joblib:master Dec 19, 2016
@lesteve lesteve deleted the allow-backend-instance-in-parallel-constructor branch December 19, 2016 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@lesteve