Lightning-AI/pytorch-lightning

Adopt `typing_extensions.Override`

Open

#18.695 aperta il 3 ott 2023

Vedi su GitHub
 (25 commenti) (0 reazioni) (1 assegnatario)Python (3233 fork)batch import
featuregood first issuehelp wanted

Metriche repository

Star
 (26.687 star)
Metriche merge PR
 (Merge medio 9g 15h) (3 PR mergiate in 30 g)

Descrizione

Description & Motivation

We use overrides heavily.

Python 3.12 added support for this. See https://peps.python.org/pep-0698/

Mypy should support it: https://github.com/python/mypy/issues/14072

Pitch

Use https://typing-extensions.readthedocs.io/en/latest/#override in our overrides classes.

Using this would require increasing the minimum typing_extensions to 4.4: https://github.com/Lightning-AI/lightning/blob/master/requirements/fabric/base.txt#L8

Alternatives

No response

Additional context

No response

cc @borda

Guida contributor

Direzione di ricerca
Identifica tutte le classi che sovrascrivono metodi dalle classi genitore e aggiungi il decoratore `@typing extensions.override`. Assicurati di aggiornare la versione minima di `typing extensions` a 4.4 nel file dei requisiti.
Tech stack
python
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65