ENH Add a switch to override existing global variables #216
Conversation
Codecov Report
@@ Coverage Diff @@
## master #216 +/- ##
========================================
+ Coverage 84.92% 85% +0.07%
========================================
Files 1 1
Lines 577 580 +3
Branches 114 115 +1
========================================
+ Hits 490 493 +3
Misses 63 63
Partials 24 24
Continue to review full report at Codecov.
|
|
@suquark |
pierreglaser
force-pushed
the
set-custom-globals-overriding-behavior
branch
from
0b818ac
to
0f49713
Compare
| if state.get('override_existing_globals', True): | ||
| func.__globals__.update(state['globals']) | ||
| else: | ||
| for k, v in state['globals'].items(): |
There was a problem hiding this comment.
I would re-add the original comment for this block of the condition:
# Only set global variables that do not exist.
pierreglaser
force-pushed
the
set-custom-globals-overriding-behavior
branch
from
06f38ba
to
f6d6337
Compare
pierreglaser
force-pushed
the
set-custom-globals-overriding-behavior
branch
from
07f8a3e
to
3f32518
Compare
|
@suquark could you please provide more details? Do you have a standalone example of one such problematic recursive function? |
|
Closing in favor of #240. |
Following #214.
This PR implements a switch called
override_existing_globals, available fromCloudPickler.__init__,cloudpickle.dumpsandcloudpickle.dump.True, the globals from a function originally created in the__main__module, dynamic modules or in a nested scope override the globals of the function's module at unpickling time.False, the existing variables in the function's module overrides the globals of those functions at unpickling time.