Unions with generic parameter reduces parameter too far
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Bug Report
When creating a method that takes a union of a type with a generic parameter it seems like the generic parameter is reduced to object instead of its original value.
To Reproduce
import typing as t
T = t.TypeVar('T', covariant=True)
Y = t.TypeVar('Y')
class Nothing(t.Generic[T]):
def chain_none(self: 'Nothing[t.Optional[Y]]') -> 'Nothing[Y]':
return self # type: ignore
class Just(t.Generic[T]):
def __init__(self, val: T) -> None:
self._value = val
def chain_none(
self: 'Just[t.Optional[Y]]') -> t.Union[Nothing[Y], 'Just[Y]']:
if self._value is None:
return Nothing()
return Just(self._value)
val: t.Union[Nothing[t.Union[None, int, str]], Just[t.Union[None, int, str]]]
reveal_type(val.chain_none())
This reveals 'Union[mypy_test.Nothing[builtins.object], mypy_test.Just[builtins.object]]' while I would expect 'Union[mypy_test.Nothing[Union[int, str]], mypy_test.Just[Union[int, str]]].
Please note that the example does work correctly when using Union[None, int] for val.
Your Environment
- Mypy version used: 0.790
- Python version used: 3.8.5
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
Führe die bereitgestellte Python-3.8.5-Reproduktion mit mypy 0.790 aus und vergleiche die Ausgabe von reveal_type mit der erwarteten Union. Verfolge die Reduktion des generischen Parameters für den Union-Fall und füge anschließend Tests hinzu, die zeigen, dass das Ergebnis Union[int, str] beibehält, statt es auf object zu reduzieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100