facebookresearch/metaseq

Convert string matches to enum comparison if possible

Open

#311 ouverte le 20 août 2022

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Python (701 forks)batch import
better-engconfigenhancementgood first issue

Métriques du dépôt

Stars
 (6 195 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Recherchez les comparaisons de chaînes littérales en utilisant `ag "== \"" py` dans le dépôt, puis identifiez les chaînes qui correspondent à des valeurs d'énumération connues. Définissez ou réutilisez des énumérations existantes (par exemple, depuis le module enum de Python) et remplacez les vérifications de chaînes par des comparaisons d'énumération. Assurez vous que toutes les occurrences sont converties et que les tests passent.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85