Skip to content

Unvendoring joblib#2019

Closed
GaelVaroquaux wants to merge 1 commit intodask:masterfrom
GaelVaroquaux:unvendoring_joblib
Closed

Unvendoring joblib#2019
GaelVaroquaux wants to merge 1 commit intodask:masterfrom
GaelVaroquaux:unvendoring_joblib

Conversation

@GaelVaroquaux
Copy link
Copy Markdown

Follows scikit-learn/scikit-learn#11166 to deal with unvendoring joblib in scikit-learn

Follows scikit-learn/scikit-learn#11166 to deal
with unvendoring joblib in scikit-learn
try:
from sklearn.externals.joblib.parallel import ( # noqa
AutoBatchingMixin, ParallelBackendBase)
except ModuleNotFoundError:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Guess we could add the following for Python 2/3 support

try:
    ModuleNotFoundError
except NameError:
    ModuleNotFoundError = ImportError

@jakirkham
Copy link
Copy Markdown
Member

Should this be closed in light of PR ( #2298 )?

@mrocklin
Copy link
Copy Markdown
Member

mrocklin commented Nov 3, 2018 via email

@mrocklin mrocklin closed this Apr 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants