Incorrect handling of unions of generics
Abierto
Nadie ha tomado este issue todavía.
bug
topic-type-variables
topic-union-types
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
This was originally reported in https://github.com/python/mypy/issues/3644#issuecomment-825494986
from typing import Callable, TypeVar, Generic, Union
_U = TypeVar('_U')
_T = TypeVar('_T')
class Future(Generic[_T]):
def __init__(self, t: _T):
self._t = t
def then(self, fn: Callable[[_T], Future[_U]]) -> Future[_U]:
return fn(self._t)
def then2(self, fn: Callable[[_T], _U]) -> Future[_U]:
return Future(fn(self._t))
def then3(self, fn: Union[Callable[[_T], Future[_U]], Callable[[_T], _U]]) -> Future[_U]:
ret = fn(self._t)
if isinstance(ret, Future):
return ret
else:
return Future(ret)
def foo(t: int) -> Future[str]:
return Future("str")
fut = Future(5)
reveal_type(fut.then(foo)) # is a Future[str], good
def foo2(t: int) -> str:
return "str"
reveal_type(fut.then2(foo2)) # also is a Future[str], good
reveal_type(fut.then3(foo)) # <<==== error, Future[<nothing>]
reveal_type(fut.then3(foo2))
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.
Línea de trabajo
Empieza ejecutando el reproductor de Python proporcionado con mypy y comprobando los resultados de reveal_type para then3. Rastrea la inferencia para la Union de tipos invocables y considera el issue completo cuando tanto foo como foo2 sean aceptados y se infiera Future[str] en lugar de Future[nothing].
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers, devtools
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100