Skip to content

DOC serialization and processes#803

Merged
ogrisel merged 4 commits intojoblib:masterfrom
tomMoral:DOC_loky_pickler
Nov 4, 2018
Merged

DOC serialization and processes#803
ogrisel merged 4 commits intojoblib:masterfrom
tomMoral:DOC_loky_pickler

Conversation

@tomMoral
Copy link
Copy Markdown
Contributor

@tomMoral tomMoral commented Nov 3, 2018

Add explanation on the serialization process in joblib and also add an example introducing set_loky_pickler, LOKY_PICKLER and wrap_non_picklable_objects.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 3, 2018

Codecov Report

Merging #803 into master will increase coverage by 0.61%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #803      +/-   ##
==========================================
+ Coverage   94.64%   95.25%   +0.61%     
==========================================
  Files          44       44              
  Lines        6308     6308              
==========================================
+ Hits         5970     6009      +39     
+ Misses        338      299      -39
Impacted Files Coverage Δ
joblib/_parallel_backends.py 93.6% <0%> (-1.21%) ⬇️
joblib/test/test_memory.py 98.11% <0%> (+0.15%) ⬆️
joblib/test/test_memmapping.py 99.41% <0%> (+0.29%) ⬆️
joblib/test/test_parallel.py 96.85% <0%> (+0.71%) ⬆️
joblib/func_inspect.py 95.45% <0%> (+1.13%) ⬆️
joblib/logger.py 86.84% <0%> (+1.31%) ⬆️
joblib/disk.py 81.66% <0%> (+1.66%) ⬆️
joblib/test/common.py 88.13% <0%> (+1.69%) ⬆️
joblib/pool.py 91.37% <0%> (+1.72%) ⬆️
joblib/test/test_store_backends.py 97.05% <0%> (+5.88%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f1ca52...2ac1feb. Read the comment docs.

Copy link
Copy Markdown
Contributor

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

LGTM, just a few fixes. Thanks very much @tomMoral!

@ogrisel ogrisel merged commit 30a2edd into joblib:master Nov 4, 2018
@ogrisel
Copy link
Copy Markdown
Contributor

ogrisel commented Nov 4, 2018

Merged! Thanks @tomMoral.

@tomMoral tomMoral deleted the DOC_loky_pickler branch November 6, 2018 10:08
@albertcthomas
Copy link
Copy Markdown
Contributor

Thanks a lot @tomMoral !!

yarikoptic added a commit to yarikoptic/joblib that referenced this pull request Apr 12, 2019
* tag '0.13.0':
  Release 0.13.0
  Use https (joblib#805)
  MTN bump loky to 2.4.2 (joblib#804)
  DOC: provide some useful variables for Makefile (joblib#794)
  DOC serialization and processes (joblib#803)
  ENH update loky 2.4.0 (joblib#802)
  FIX backward compatibility for nested backend (joblib#789)
  enable python 3.7 (joblib#795)
  memory: add test for call_and_shelve performance (joblib#791)
  FIX nested backend not changed by SequentialBackend (joblib#792)
  cloudpickle 0.6.0 (joblib#788)
  FIX nested backend setting n_jobs=-1 (joblib#785)
  Raises a more explicit exception when a corrupted MemorizedResult is loaded (joblib#768)
  Back to dev mode
yarikoptic added a commit to yarikoptic/joblib that referenced this pull request Apr 12, 2019
* releases:
  Release 0.13.0
  Use https (joblib#805)
  MTN bump loky to 2.4.2 (joblib#804)
  DOC: provide some useful variables for Makefile (joblib#794)
  DOC serialization and processes (joblib#803)
  ENH update loky 2.4.0 (joblib#802)
  FIX backward compatibility for nested backend (joblib#789)
  enable python 3.7 (joblib#795)
  memory: add test for call_and_shelve performance (joblib#791)
  FIX nested backend not changed by SequentialBackend (joblib#792)
  cloudpickle 0.6.0 (joblib#788)
  FIX nested backend setting n_jobs=-1 (joblib#785)
  Raises a more explicit exception when a corrupted MemorizedResult is loaded (joblib#768)
  Back to dev mode
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