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.

Sign up for GitHub

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

Jump to bottom

Support PyCapsule #477

Merged

mmckerns merged 5 commits into uqfoundation:master from anivegesana:pycapsule

Jun 7, 2022

Contributor

anivegesana commented May 17, 2022

Fixes #354


          Support PyCapsule

44095bc

anivegesana force-pushed the pycapsule branch 3 times, most recently from 03c7497 to 361f048 Compare

May 17, 2022 07:20


          Import PyCapsule if it already exists and add descriptive warnings

6fff6d8

anivegesana force-pushed the pycapsule branch from 361f048 to 6fff6d8 Compare

May 17, 2022 07:37

mmckerns added this to the dill-0.3.6 milestone

mmckerns self-requested a review

June 5, 2022 09:45

mmckerns mentioned this pull request

With NumPy 1.20, SymPy generated code cannot be serialized with dill #405

Open

mmckerns reviewed

View reviewed changes

dill/_dill.py Outdated Show resolved Hide resolved

mmckerns reviewed

View reviewed changes

tests/test_objects.py Outdated Show resolved Hide resolved

mmckerns added the enhancement label

mmckerns reviewed

View reviewed changes

dill/_dill.py Show resolved Hide resolved

mmckerns reviewed

View reviewed changes

Member

mmckerns left a comment

Minor adjustments needed here, otherwise LGTM.

anivegesana added 2 commits

June 6, 2022 14:05


          Merge remote-tracking branch 'uqfoundation/master' into pycapsule

5d18380


          Suggestions from code review

c356174

mmckerns reviewed

View reviewed changes

tests/test_pycapsule.py Show resolved Hide resolved

mmckerns approved these changes

View reviewed changes

Member

mmckerns left a comment •

edited

Loading

LGTM. I'll give you some time to change the Author and Copyright on the new test file. I'll pull the PR in the morning.


          Update copyright

960a846

mmckerns merged commit 74de347 into uqfoundation:master

mmckerns mentioned this pull request

dill fails to pickle sympy.Float under Python 3.7 for dill.settings["recurse"]=True #543

Closed

jbschlosser mentioned this pull request

Cannot serialize NJTs using torch.save pytorch/pytorch#129366

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels