joblib / loky

Robust and reusable Executor for joblib
http://loky.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
520 stars 47 forks source link

CLN remove backward compat and rely on multiprocessing for win reduction #394

Closed tomMoral closed 1 year ago

ogrisel commented 1 year ago

The windows py38 deadlock does not appear on master. So maybe this backward compat is required at least for Python 3.8?

The py311 failures are already present on master and are therefore unrelated to this PR.

tomMoral commented 1 year ago

The windows py38 deadlock does not appear on master. So maybe this backward compat is required at least for Python 3.8?

The py38 failure on the first commit is on macos so I don't think it is related to this PR in fact, I think we both misread this :sweat_smile:

ogrisel commented 1 year ago

@tomMoral we need to document the change in the changelog. In particular, I think this is a bugfix for nested use of loky under Python 3.11 (compared to the last released version).