-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[ENH] make parallelisation default option wherever applicable #6387
Comments
That is not universally true - in cases where Another question, is parallelization with Two arguments against:
For me, the second argument makes the "contra" side outweigh the "pro" side at the moment, of course I'm happy to listen to arguments and weightings by others, as I do not think my own assessment is fully consolidated here, or complete. |
@fkiraly |
Yes, that one - it is precisely an example of the case in the 2nd bullet point, with the inner estimator being |
Currently all (as far as I know) parallelisation options are disabled by default and needs to be enabled using set_config. But almost always end users will prefer a parallel version over sequential one.
This issue to propose to make the parallelisation by
loky
with joblib as the default option.If users prefer to disable, for debugging or other purposes, they should have the capability using set_config.
If users prefer parallelisation by other tools, e.g. dask, they should be given the option as well, but default should be based on a method that's always available through core dependencies.
The text was updated successfully, but these errors were encountered: