facebookresearch/ParlAI

Unify names of some options

Open

#2 170 ouverte le 18 nov. 2019

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Python (2 089 forks)batch import
Code QualityH1 2020Help WantedMinordonotreap

Métriques du dépôt

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

Description

Now that we have upgrade_opt we can do some resolution of the embedding_size and embeddingsize madness we have in seq2seq vs transformer vs memnn etc.

Guide contributeur

Direction de recherche
Recherchez dans la base de code toutes les occurrences de 'embedding size' et 'embeddingsize'. Celles ci apparaissent dans les agents seq2seq, transformer et memnn. Utilisez le mécanisme existant 'upgrade opt' (probablement dans options.py ou similaire) pour créer un alias des anciens noms incohérents vers un nom unifié unique. Assurez vous que tous les fichiers de configuration existants continuent de fonctionner en mappant les anciens noms. Consultez les commentaires de l'issue pour toute orientation supplémentaire sur le nom unifié préféré.
Stack technique
pythonpytorch
Domaine
developer experience
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonParlAI codebase familiarityunderstanding of option naming conventions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65