Callable with Iterator[T] return type and default function with compatible return type, errors
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Bug Report
I am trying to add some type hints to a module that takes a filename and reads it with a caller supplied reader (most common csv.reader and csv.DictReader). I noticed when I do not supply a default, mypy is fine, however, when I do, it complains about incompatibility.
To Reproduce
import csv
from typing import Callable, Iterable, Iterator, Text, TypeVar
T = TypeVar('T')
def foo(x: str, reader: Callable[[Iterable[Text]], Iterator[T]]) -> Iterator[T]:
return reader(x)
# Is fine, resolves to Iterator[List[str]]
x = foo('bar.txt', csv.reader)
# Complains that _reader is not compatible with Iterator[T]
def baz(x: str, reader: Callable[[Iterable[Text]], Iterator[T]] = csv.reader) -> Iterator[T]:
return reader(x)
When I look at the _reader definition I see
class _reader(Iterator[List[str]]):
dialect: Dialect
line_num: int
if sys.version_info >= (3, 0):
def __next__(self) -> List[str]: ...
else:
def next(self) -> List[str]: ...
So it is a subclass of Iterator[List[str]] which should be compatible with Iterator[T]
Expected Behavior
No mypy error is shown
Actual Behavior
mypy complains about using csv.reader as the default value of the reader argument due to it not matching _reader.
Your Environment
- Mypy version used: 0.812
- Mypy command-line flags:
- Mypy configuration options from
mypy.ini(and other config files):
[mypy]
# Most copied from https://github.com/python/mypy/blob/675ae1afc8a5b0a9a515292c7e9d6bc2a38887be/mypy_self_check.ini
disallow_untyped_calls = True
disallow_untyped_defs = True
disallow_incomplete_defs = True
check_untyped_defs = True
disallow_subclassing_any = True
warn_no_return = True
strict_optional = True
strict_equality = True
no_implicit_optional = True
disallow_any_generics = True
disallow_any_unimported = True
warn_redundant_casts = True
warn_unused_ignores = True
warn_unused_configs = True
show_traceback = True
show_error_codes = True
# Should be removed if we use typehints everywhere
follow_imports = silent
python_version = 3.9
[mypy-memory_profiler.*]
ignore_missing_imports = True
- Python version used: 3.9.1
- Operating system and version: Ubuntu 20.10
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia eseguendo lo snippet Python fornito con mypy 0.812 e confronta il parametro reader generico in foo con il valore predefinito in baz. Traccia il modo in cui mypy verifica un callable predefinito rispetto a un tipo di ritorno generico. Il lavoro è completato quando baz accetta csv.reader senza un errore di incompatibilità, preservando il comportamento esistente di foo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100