Mixin pattern does not play well with slots
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
Direzione di ricerca
Inizia con la riproduzione fornita di HasTimestampMixin e ConcreteMessage, quindi esegui mypy e confronta il suo risultato con pyright e ty. Traccia il modo in cui mypy verifica le assegnazioni ad attributi annotati attraverso slots e la MRO; il lavoro è completato quando l’esempio supera il controllo dei tipi senza l’errore segnalato, preservando al contempo il comportamento degli slot della classe concreta.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I am annotating a large codebase with heavy use of mixin patterns, where classes sharing similar attributes inherit the mixin for additional functionality.
One pattern in particular is failing on mypy (but passes on pyright and ty): where slots are involved in the concrete class. Let's look at this MRO, where HasTimestampMixin offers another way of setting the timestamp attribute.
from datetime import datetime
class HasTimestampMixin:
__slots__ = []
# Attribute contract: concrete subclasses provide this slot.
timestamp: str | None
@property
def timestamp_dt(self) -> datetime | None:
if self.timestamp is None:
return None
return datetime.fromisoformat(self.timestamp)
@timestamp_dt.setter
def timestamp_dt(self, dt: datetime | None) -> None:
# this line fails with
# Trying to assign name "timestamp" that is not in "__slots__" of type "repro.HasTimestampMixin" [misc]
self.timestamp = None if dt is None else dt.isoformat()
class ConcreteMessage(HasTimestampMixin):
__slots__ = ["timestamp"]
timestamp: str | None
def __init__(self) -> None:
self.timestamp = None
message = ConcreteMessage()
message.timestamp_dt = datetime(2026, 7, 30)
print(message.timestamp) # Output: 2026-07-30T00:00:00
Removing the __slots__ in the mixin class fixes the typing issue but breaks the runtime by no longer having slots in the concrete class.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Merge medio
- 1g 12h
- PR unite (30g)
- 58
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.
Altre issue di python/mypy
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
bug topic-configuration topic-error-reporting
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
bug topic-attrs
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
anthropics/skills#1811 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
speaches-ai/speaches#678 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
datalayer/mcp-compose#42 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
conda-forge/spacy-feedstock#177 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
UKGovernmentBEIS/inspect_evals#2523 ·