facebookresearch/ParlAI

ParlaiParser.add_argument should support no_save and no_override

Ouverte

#2 177 ouverte le 19 nov. 2019

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Python (2 089 forks)batch import
EnhancementH1 2020Help WantedMinordonotreap

Métriques du dépôt

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

Description

Depends on #2172.

  • If no_save is True, then the option is not saved with the opt.
  • If no_override is True, then the parser will throw a hard error if the value on disk is not what's loaded. This one might have to wait a bit longer.

Guide contributeur

Direction de recherche
Etudiez l'implementation actuelle de ParlaiParser et l'issue referencee #2172. Ajoutez les parametres 'no save' et 'no override' a la methode add argument, en veillant a ce que 'no save' empeche l'option d'etre sauvegardee dans opt, et 'no override' leve une erreur si la valeur sur le disque differe. Ecrivez des tests pour verifier le comportement.
Stack technique
python
Domaine
developer experiencetooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70