classification
Title: inspect.getclosurevars() does not get all globals
Type: behavior Stage:
Components: Library (Lib) Versions: Python 3.7
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: azag0, josh.r
Priority: normal Keywords:

Created on 2018-10-09 20:58 by azag0, last changed 2018-10-09 23:04 by josh.r.

Messages (2)
msg327436 - (view) Author: Jan Hermann (azag0) Date: 2018-10-09 20:58
inspect.getclosurevars() omits globals or nonlocals that are bound within comprehensions:

22:50 ~ python3
Python 3.7.0 (default, Aug 17 2018, 21:14:48) 
[Clang 9.1.0 (clang-902.0.39.2)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
(ins)>>> import inspect
(ins)>>> x = 1
(ins)>>> def f():
(ins)...     return [x for _ in range(1)]
(ins)... 
(ins)>>> inspect.getclosurevars(f)
ClosureVars(nonlocals={}, globals={}, builtins={'range': <class 'range'>}, unbound=set())
(ins)>>> def f():
(ins)...     return x
(ins)... 
(ins)>>> inspect.getclosurevars(f)
ClosureVars(nonlocals={}, globals={'x': 1}, builtins={}, unbound=set())

It can be fixed quite easily along the following lines:

https://github.com/azag0/calcfw/blob/master/caf2/hashing/func.py#L133-L146

Does this look like a reasonable solution?
msg327444 - (view) Author: Josh Rosenberg (josh.r) * Date: 2018-10-09 23:04
Problem: The variables from the nested functions (which comprehensions are effectively a special case of) aren't actually closure variables for the function being inspected.

Allowing recursive identification of all closure variables might be helpful in some contexts, but you wouldn't want it to be the only behavior; it's easier to convert a non-recursive solution to a recursive solution than the other way around.
History
Date User Action Args
2018-10-09 23:04:58josh.rsetnosy: + josh.r
messages: + msg327444
2018-10-09 20:58:00azag0create