-
Notifications
You must be signed in to change notification settings - Fork 1.3k
replace array.__reduce__
to array.__reduce_ex__
#3876
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
Comments
reduce_ex
Although the #3611 issue mentions the implementation of |
reduce_ex
array.__reduce__
to array.__reduce_ex__
This is likely why @Snowapril mentioned
What methods does
|
@Yaminyam We need to separate definition of
Thus, if we don't need protocol version for implementing It seems true that we don't have to define |
@qingshi163 implemented @maong0927 Thanks for bringing this to our attention. As @Snowapril points out, the fix seems simple. |
well, working is better than broken. I think we are going on correct way including #3064 |
Feature
In rustPython's array pickling,
reduce
is used.However,
reduce_ex
is used in cPython's pickle.I think we should implement
reduce_ex
in the array and usereduce_ex
for picking.Execution Result
There is no related python documentation, so I will attach the execution result.

The text was updated successfully, but these errors were encountered: