Overeager type narrowing
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
I'm implementing type narrowing and thinking about intersection types for Python in general and I noticed this behavior in mypy:
Point = tuple[int, int]
class Shape: pass
def scale(x: float, o: int | Point | Shape):
if isinstance(o, tuple):
reveal_type(o)
Will show Tuple[builtins.int, builtins.int] which might seem reasonable but what if there's
class ColorShape(Shape, tuple[Shape, str]): pass
We should say that the intersection of Point and tuple (= tuple[Any, ...]) is Point (= tuple[int, int]). The intersection of Shape and tuple is not known to be empty.
The intersection of int and tuple is known to be empty for CPython because:
>>> class A(int, tuple): pass
...
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: multiple bases have instance lay-out conflict
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 l’esempio inline di mypy e controllando il tipo rivelato per il ramo isinstance(o, tuple). Traccia il comportamento del restringimento dei tipi per i membri dell’unione Point, Shape e int; il lavoro è completato quando il restringimento preserva le possibili intersezioni di tuple senza trattare i tipi non correlati o potenzialmente compatibili come impossibili.
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