Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

[WIP] FIX CDClassifier with n_jobs != 1 in py36 (#114) #117

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

vene
Copy link
Contributor

@vene vene commented Jun 10, 2017

However, the change breaks the test test_debiasing_l1l2 on my box, I can't tell why.

@vene vene requested review from mblondel and fabianp June 10, 2017 06:29
@vene
Copy link
Contributor Author

vene commented Jun 10, 2017

(rebased onto #118 to benefit from actually relevant continuous integration)

@vene vene force-pushed the cdefpkl branch 2 times, most recently from 178d48a to f06f871 Compare June 10, 2017 09:28
Copy link
Member

@fabianp fabianp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, nothing to add :-)

@aldanor
Copy link
Contributor

aldanor commented Mar 28, 2018

@fabianp @mblondel Could this get possibly merged?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants