-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
statistics.NormalDist
fails to be pickled with 0
and 1
protocols
#99155
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
sobolevn
added a commit
to sobolevn/cpython
that referenced
this issue
Nov 6, 2022
I've applied this to |
vfazio
pushed a commit
to vfazio/cpython
that referenced
this issue
Nov 7, 2022
vfazio
pushed a commit
to vfazio/cpython
that referenced
this issue
Nov 7, 2022
Yes, sure! |
sobolevn
added a commit
to sobolevn/cpython
that referenced
this issue
Nov 7, 2022
…cols (pythonGH-99156). (cherry picked from commit d7a00f1) Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
sobolevn
added a commit
to sobolevn/cpython
that referenced
this issue
Nov 7, 2022
…cols (pythonGH-99156). (cherry picked from commit d7a00f1) Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Nov 7, 2022
…cols (pythonGH-99156). (pythonGH-99188) (cherry picked from commit d7a00f1) Co-authored-by: Nikita Sobolev <mail@sobolevn.me> (cherry picked from commit 263e983) Co-authored-by: Nikita Sobolev <mail@sobolevn.me> Automerge-Triggered-By: GH:rhettinger
rhettinger
pushed a commit
that referenced
this issue
Nov 7, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Repro:
This is actually tested in
test_statistics
like this:cpython/Lib/test/test_statistics.py
Lines 3012 to 3013 in 8feb7ab
But, it is only testing default protocol and is missing
for proto in range(pickle.HIGHEST_PROTOCOL + 1):
part. Note: this is the only place in all of the CPython tests, whereprotocol=
is not passed. All other places do have the full check.It looks like a bug to me and I will send a PR with the fix.
CC @rhettinger
NormalDist
pickle with0
and1
protocols #99156NormalDist
pickle with0
and1
protocols (GH-99156). #99188NormalDist
pickle with0
and1
protocols (GH-99156). (GH-99188) #99190The text was updated successfully, but these errors were encountered: