Type refinement does not work with `None in [a, b, c]`, `any()`, or `all()` checks
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza con los ejemplos reproducibles para las comprobaciones de pertenencia de None y las comprobaciones de all() y, después, sigue el manejo del refinamiento de tipos de mypy para estas expresiones condicionales. Se considera terminado cuando las variables indicadas se han reducido de Optional[str] y Optional[int] después de las comprobaciones, sin los errores mostrados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When checking if multiple variables are not None using the syntax None in [a, b, c], mypy fails to refine the types of these variables to non-Optional types after this check. This issue persists even when using any() or all() for similar checks, preventing mypy from correctly inferring that the variables cannot be None past these checks, despite the code logically ensuring that these variables are not None.
Code to Reproduce:
from typing import Optional
def init_vars(
a: Optional[str] = None,
b: Optional[int] = None,
c: Optional[int] = None,
) -> None:
if None in [a, b, c]:
raise ValueError("a, b, and c cannot be None")
# Expected: a, b, c should not be Optional types beyond this point
# Actual: mypy still treats a, b, c as Optional types
print(a.upper()) # mypy error: Item "None" of "Optional[str]" has no attribute "upper"
print(b + 1) # mypy error: Unsupported operand type(s) for +: 'int' and 'NoneType'
print(c + 1) # mypy error: Unsupported operand type(s) for +: 'int' and 'NoneType'
And here is another example with the all function (with any it is quite similar):
def init_vars_with_all(
a: Optional[str] = None,
b: Optional[int] = None,
c: Optional[int] = None,
) -> None:
if not all(x is not None for x in [a, b, c]):
raise ValueError("a, b, and c cannot be None")
# Expected: a, b, c should not be Optional types beyond this point
# Actual: mypy still treats a, b, c as Optional types
print(a.upper()) # mypy error: same as above
print(b + 1) # mypy error: same as above
print(c + 1) # mypy error: same as above
Expected Behavior:
After the checks if None in [a, b, c]: or using all(), mypy should refine the types of a, b, and c to str, int, and int respectively, acknowledging that they cannot be None.
Actual Behavior:
mypy does not refine the types and continues to treat a, b, and c as Optional[str], Optional[int], and Optional[int] respectively. This results in type errors when attempting to use these variables in a context that does not allow None.
Additional Information:
mypy version: 1.9.0 compiled
Python version: 3.10
No flags or configurations are altering the default behavior of mypy in this instance.
This behavior suggests a limitation in mypy's type inference system when it comes to using collective None checks with list membership or aggregate functions like any() and all(). An enhancement to allow type refinement in such cases would significantly improve usability in scenarios involving initialization checks for multiple variables.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Merge medio
- 1 d 18 h
- PR fusionados (30 d)
- 54
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de python/mypy
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
bug topic-configuration topic-error-reporting
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Todos los issues de python/mypy
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
bancolombia/sentinel#23 ·
-
test md AbiertoCI
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
langchain-ai/deepagents#6450 ·
-
bug client
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100