-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
FIX propagate configuration to workers in parallel #25363
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
Changes from all commits
c9fcaa1
a10e6da
790baa1
3ed8832
c9b5d02
ef3ef2f
9b2eb17
a1fb2be
689792f
5b3c9a1
4a4eee4
4af933b
ea03b3e
7676555
90ce7d1
b80d5df
a25add1
eddea10
1fd6a88
4015591
358e33e
32cda57
a5c83a8
2b92410
b16203e
d2fb6e6
8f9e99f
19661ee
4f7bd37
9320a51
dbdb2cd
1bd78b7
15526e8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,6 +9,16 @@ Version 1.2.1 | |
|
||
**In Development** | ||
|
||
Changes impacting all modules | ||
----------------------------- | ||
|
||
- |Fix| Fix a bug where the current configuration was ignored in estimators using | ||
`n_jobs > 1`. This bug was triggered for tasks dispatched by the auxillary | ||
thread of `joblib` as :func:`sklearn.get_config` used to access an empty thread | ||
local configuration instead of the configuration visible from the thread where | ||
`joblib.Parallel` was first called. | ||
:pr:`25363` by :user:`Guillaume Lemaitre <glemaitre>`. | ||
|
||
Changed models | ||
-------------- | ||
|
||
|
@@ -105,6 +115,13 @@ Changelog | |
boolean. The type is maintained, instead of converting to `float64.` | ||
:pr:`25147` by :user:`Tim Head <betatim>`. | ||
|
||
- |API| :func:`utils.fixes.delayed` is deprecated in 1.2.1 and will be removed | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we need a changelog entry to introduce There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Rather than a new entry I think we can just expand this entry as suggested below to keep related changes together: |
||
in 1.5. Instead, import :func:`utils.parallel.delayed` and use it in | ||
conjunction with the newly introduced :func:`utils.parallel.Parallel` | ||
to ensure proper propagation of the scikit-learn configuration to | ||
the workers. | ||
:pr:`25363` by :user:`Guillaume Lemaitre <glemaitre>`. | ||
|
||
.. _changes_1_2: | ||
|
||
Version 1.2.0 | ||
|
Uh oh!
There was an error while loading. Please reload this page.