-
-
Notifications
You must be signed in to change notification settings - Fork 12k
[Deprecated] ENH: enable pickle protocol 5 support for python3.5 #16421
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
Conversation
|
Seems reasonable. If we want to put this in, it has to be in the next few days, since it is only useful for 1.19, after that 3.5 will be dropped and there is no further 1.18 release planned right now. |
|
Python 3.5 is already dropped in 1.19, we couldn't support f-strings otherwise :) It is too bad that the 3.5 backport wasn't available earlier. If you really, really need it for 3.5 you might try patching a 1.18 source release and compiling yourself. |
|
If you want to make this PR against the 1.18 branch I'll put it in just so it is there, but I don't expect to make another release of that branch. Could maybe make a source release if that would be helpful. |
|
Oops, I did't know numpy 1.19 has dropped Python3.5 support. It is totally ok for me to put it in the 1.18 branch (even without another release). |
|
@charris right. I think the release notes may need to be clarified a bit. It says |
Got it, thanks. |
|
@suquark happy to put this in, there is one test related to importerror when using protocol 5 but its not available. Could you modify that as well just to be sure? |
|
@jakirkham The PR should be against @suquark You shouldn't be working in the master branch anyway. For two commits I'd do something like this: Which will clean up your master branch. Then push your new branch and open a PR. |
|
Sorry Charles was meaning how to fix the release notes. It seems Siyuan already has this fix in hand. |
NumPy 1.19.0 is still in the rc stage, so the release notes aren't finalized yet. I'll take care of it in the rc2. |
Since the pickle5 library has enabled pickle protocol 5 support for python3.5 (https://github.com/pitrou/pickle5-backport, pitrou/pickle5-backport#15), it is reasonable to relax the constraints to support pickle protocol 5 under python3.5.