facebookresearch/ParlAI

Unify names of some options

Open

#2170 aperta il 18 nov 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (2089 fork)batch import
Code QualityH1 2020Help WantedMinordonotreap

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca nel codebase tutte le occorrenze di 'embedding size' e 'embeddingsize'. Queste compaiono negli agenti seq2seq, transformer e memnn. Utilizza il meccanismo esistente 'upgrade opt' (probabilmente in options.py o simile) per creare un alias dei vecchi nomi inconsistenti verso un unico nome unificato. Assicurati che eventuali file di configurazione esistenti continuino a funzionare mappando i vecchi nomi. Consulta i commenti dell'issue per eventuali indicazioni aggiuntive sul nome unificato preferito.
Tech stack
pythonpytorch
Dominio
developer experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
PythonParlAI codebase familiarityunderstanding of option naming conventions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65