Repository navigation
Fix unpickling a curried function defined in __main__ - #645
Open
Cherith1222 wants to merge 1 commit into
Open
Cherith1222 wants to merge 1 commit into
Cherith1222 wants to merge 1 commit into
Conversation
cloudpickle stored a __main__ name, so a fresh process could not find the function. When the module name is __main__, curry.__reduce__ keeps the function object. AI-assisted (Cursor). The account owner confirmed this can be sent after the Factory checks.
This branch has not been deployed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This patch was drafted with assistance from Cursor. The account owner confirmed it could be sent after the Factory checks (repository policy, no covering pull request, and a local test that failed first and then passed).
A
currydefined in__main__is stored by cloudpickle as__main__:f. A new process then raisesAttributeErrorbecause that name is not on the fresh__main__module.curry.__reduce__now keeps the function object when the module name is__main__.Fixes #606