isinstance check widens type
Offen
Dieses Issue hat noch niemand übernommen.
bug
topic-type-narrowing
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
- 2025-02-14: reproduced using mypy 1.15 and python 3.13 mypy-playground
- 2023-02-14: reproduced using mypy 1.0 and python 3.11 mypy-playground
from collections.abc import Collection, Mapping
def f(x: int | Collection[int] | Mapping[str, int]) -> int:
r"""Recursively sum up all the values of possibly nested data."""
if isinstance(x, int):
return x
if isinstance(x, Mapping):
return sum(f(y) for y in x.values())
reveal_type(x) # <- Here, mypy thinks this is Collection only!
if isinstance(x, Collection):
reveal_type(x) # <- Suddenly mypy thinks this is Collection | Mapping
return sum(f(y) for y in x) # ❌ "f" has incompatible type "int | str"
raise TypeError(f"unsupported type: {type(x)}")
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, den bereitgestellten Reproducer in den verknüpften mypy-playground-Beispielen mit den aufgeführten mypy- und Python-Versionen auszuführen, und verfolge dann, wie die isinstance-Prüfungen x nach dem Mapping-Zweig eingrenzen. Als erledigt gilt die Aufgabe, wenn der Collection-Zweig x konsistent eingrenzt und der gezeigte Aufruf von f die Typprüfung ohne den inkompatiblen int | str-Fehler besteht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- compilers, devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100