ENH use cloudpickle to pickle functions and class from __main__#677
Merged
tomMoral merged 13 commits intojoblib:masterfrom May 22, 2018
Merged
ENH use cloudpickle to pickle functions and class from __main__#677tomMoral merged 13 commits intojoblib:masterfrom
tomMoral merged 13 commits intojoblib:masterfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## master #677 +/- ##
=========================================
+ Coverage 95.08% 95.2% +0.12%
=========================================
Files 39 39
Lines 5427 5462 +35
=========================================
+ Hits 5160 5200 +40
+ Misses 267 262 -5
Continue to review full report at Codecov.
|
Contributor
Author
|
The failure in CI should be fixed with #685 . I noticed today that the pickling of function in main might be very long compared to the pickling of a function in another module. It can drastically degrade the performance for very large number of small tasks. I think this should be benchmarked and documented in this PR to make sure we are not introducing bottleneck in the computations. |
ogrisel
requested changes
May 22, 2018
joblib/parallel.py
Outdated
| return self._size | ||
|
|
||
| @staticmethod | ||
| def _wrap_non_picklable_objects(obj): |
Contributor
There was a problem hiding this comment.
Please insert a comment explaining why we do this shallow introspection of objects instead of using the cloudpickle.Pickler class directly to pickle everything.
# The following does a shallow introspection of the Python objects passed as
# args to a parallel call. Interactively defined functions are pre-pickled manually
# using cloudpickle.
# We do not use cloudpickle for all arguments as it would incur a significant
# performance overhead when pickling large cardinality dicts or lists for
# instance.
joblib/parallel.py
Outdated
| def _wrap_non_picklable_objects(obj): | ||
| need_wrap = "__main__" in getattr(obj, "__module__", "") | ||
| if callable(obj): | ||
| # Need wrap if the object is a local function |
Contributor
There was a problem hiding this comment.
Can you please explain what a "local function" is?
joblib/test/test_parallel.py
Outdated
| print(Parallel(n_jobs=2, backend=backend)( | ||
| delayed(run)(square, i) for i in range(5))) | ||
| print(Parallel(n_jobs=2, backend=backend)( | ||
| delayed(run)(f=square, x=i) for i in range(5))) |
Contributor
There was a problem hiding this comment.
Please add a test with a lambda expression.
- This only works for shallow object, for instance this does not work on a list of functions defined in main: args=([func_main_1, func_main_2,..],)
- Test for lambda-expr, locally defined functions and function from __main__ - Remove duplicated test
ogrisel
reviewed
May 22, 2018
joblib/parallel.py
Outdated
| self.n_dispatched_batches = 0 | ||
| self.n_dispatched_tasks = 0 | ||
| self.n_completed_tasks = 0 | ||
| # Use a caching dict for callable that are pickled with cloudpickle to |
ogrisel
reviewed
May 22, 2018
joblib/parallel.py
Outdated
| # Use a caching dict for callable that are pickled with cloudpickle to | ||
| # improve performances. This cache is used only in the case of | ||
| # functions that are defined in the __main__ module, function that are | ||
| # defined locally (inside another function) and lambda expression. |
ogrisel
reviewed
May 22, 2018
joblib/parallel.py
Outdated
| self.n_completed_tasks = 0 | ||
| # Use a caching dict for callable that are pickled with cloudpickle to | ||
| # improve performances. This cache is used only in the case of | ||
| # functions that are defined in the __main__ module, function that are |
ogrisel
approved these changes
May 22, 2018
yarikoptic
added a commit
to yarikoptic/joblib
that referenced
this pull request
Jul 28, 2018
* tag '0.12': (116 commits) Release 0.12 typo typo typo ENH add initializer limiting n_threads for C-libs (joblib#701) DOC better parallel docstring (joblib#704) [MRG] Nested parallel call thread bomb mitigation (joblib#700) MTN vendor loky2.1.3 (joblib#699) Make it possible to configure the reusable executor workers timeout (joblib#698) MAINT increase timeouts to make test more robust on travis DOC: use the .joblib extension instead of .pkl (joblib#697) [MRG] Fix exception handling in nested parallel calls (joblib#696) Fix skip test lz4 not installed (joblib#695) [MRG] numpy_pickle: several enhancements (joblib#626) Introduce Parallel.__call__ backend callbacks (joblib#689) Add distributed on readthedocs (joblib#686) Support registration of external backends (joblib#655) [MRG] Add a dask.distributed example (joblib#613) ENH use cloudpickle to pickle interactively defined callable (joblib#677) CI freeze the version of sklearn0.19.1 and scipy1.0.1 (joblib#685) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix for #674
This PR introspects the
BatchCallsand pickles functions and class methods from__main__usingcloudpickle. This is one solution that we discussed with @ogrisel . However, it does not handle deep object introspection. For instance, this would fail ifargscontains a list of functions that are defined in__main__.One solution would be to use a
try..exceptwhen launching the job but I am not sure it is possible as the pickling is handled in a separate thread at least for theProcessPoolExecutor.Another solution would be to implement a custom reducer for the function but it might not be possible to fall back on default function pickling in this case.