Allow apply udf to reference global modules in cudf.pandas #15569
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.
Description
closes #15548
_replace_closurevars
creates a new function by replacing objects with their fast versions. When creating the new function, it populatesglobals
from the result ofinspect.getclosurevars
, but it don't think it comprehensively returns all the globals accessible to the function (function.__globals__
)To minimize the change, the "fast globals" are still sourced from
inspect.getclosurevars
, and those update theold_function.__globals__
when creating a new function.Checklist