-
Notifications
You must be signed in to change notification settings - Fork 184
deduplicate cloudpickle reducers. #368
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
Merged
Merged
Changes from 10 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
a860509
backport python3.8 save_reduce into old CloudPickler
pierreglaser ee722f1
python 3.8 compat
pierreglaser d69ca86
use the same Cloudpickler class for all Pythons
pierreglaser 6264641
increase flake8 max complexity
pierreglaser dc9ba84
unused functions and imports
pierreglaser d5ed4d9
pypy compat
pierreglaser c44ad88
be pickle5 friendly
pierreglaser fd67f4b
duplicate top-level functions
pierreglaser 9740b22
increase flake8 max-complexity
pierreglaser c657fe1
install tornado in the CI
pierreglaser abbb978
Update cloudpickle/cloudpickle.py
pierreglaser 9d56632
test best-effort cloudpickle backward compat
pierreglaser 9e92413
debug ci not launching
pierreglaser 043ff81
Merge branch 'master' into dedup-reducers
pierreglaser 09d9e33
solve merge conflics mistakes
pierreglaser f74372a
generate old pickle files during CI
pierreglaser 829b919
fixup! generate old pickle files during CI
pierreglaser f693401
fix pathlib/open compat
pierreglaser 46405e5
update python-nightly entry
pierreglaser df6fa9b
fix ci script
pierreglaser 20a522b
fix variable reference in ci script
pierreglaser 3908dfe
Merge branch 'master' into dedup-reducers
pierreglaser e5deaf6
restore backward compat with cloudpickle master
pierreglaser ff2baa9
add deprecation warnings inside old reconstructors
pierreglaser 13a76f9
always use up-to-date copyreg
pierreglaser 393b312
Merge branch 'dedup-reducers' of github.com:pierreglaser/cloudpickle …
pierreglaser 52746b9
don't (always) catch deprecation warnings
pierreglaser b531b14
fixup! don't (always) catch deprecation warnings
pierreglaser e85ab4d
debug ci
pierreglaser 2a82a41
fixup! debug ci
pierreglaser 21c82b9
fixup! debug ci
pierreglaser 4fb4ca5
fixup! debug ci
pierreglaser 5909c38
remove debug leftovers
pierreglaser 2822390
separate pickles per python version
pierreglaser d239943
fixup! separate pickles per python version
pierreglaser 8deef74
fixup! separate pickles per python version
pierreglaser bb3436f
add some pickles generated with cloudpickle 1.4.1
pierreglaser 884ee89
stop generating pickles inside the CI
pierreglaser 3b98492
fix use correct pypy version
pierreglaser 371d015
Remove commented lines
ogrisel d661ce6
tornado is already in dev-requirements.txt
ogrisel 4de14b2
Test nested function in old_pickles
ogrisel a0f8d73
Skip failing macos + pypy3
ogrisel ca07fce
Trigger CI
ogrisel 96fe5c0
copy cloudpickle_fast.py inside ray source
pierreglaser 78602e2
fixup! copy cloudpickle_fast.py inside ray source
pierreglaser dad2938
fixup! copy cloudpickle_fast.py inside ray source
pierreglaser 4dcbf3e
update vendored ray to more recent version
pierreglaser 677a114
install ray outside of cloudpickle
pierreglaser cd7cd1a
don't try to test deleted ray tests
pierreglaser 8eba950
fixup! don't try to test deleted ray tests
pierreglaser File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,7 @@ | ||
| from __future__ import absolute_import | ||
|
|
||
| import sys | ||
| import pickle | ||
|
|
||
|
|
||
| from cloudpickle.cloudpickle import * | ||
| if sys.version_info[:2] >= (3, 8): | ||
| from cloudpickle.cloudpickle_fast import CloudPickler, dumps, dump | ||
| from cloudpickle.cloudpickle import * # noqa | ||
| from cloudpickle.cloudpickle_fast import CloudPickler, dumps, dump # noqa | ||
|
|
||
| __version__ = '1.5.0dev0' |
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.