facebookresearch/metaseq

Convert string matches to enum comparison if possible

Open

#311 aperta il 20 ago 2022

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Python (701 fork)batch import
better-engconfigenhancementgood first issue

Metriche repository

Star
 (6195 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

We have a lot of places where we do string matching, like:

https://github.com/facebookresearch/metaseq/blob/88ae968e679efbe84a8c246d1177852facfc43a2/metaseq/tasks/streaming_language_modeling.py#L339

or

https://github.com/facebookresearch/metaseq/blob/88ae968e679efbe84a8c246d1177852facfc43a2/metaseq_cli/train.py#L96

which can all be found via: ag "== \"" --py.

Convert these all to comparing against enums if possible.

Guida contributor

Direzione di ricerca
Cerca i confronti di stringhe letterali usando `ag "== \"" py` nel repository, quindi identifica quali stringhe corrispondono a valori enum noti. Definisci o riutilizza enum esistenti (ad esempio dal modulo enum di Python) e sostituisci i controlli delle stringhe con confronti enum. Assicurati che tutte le occorrenze siano convertite e che i test passino.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85