facebookresearch/ParlAI

Unify names of some options

Open

#2.170 aberto em 18 de nov. de 2019

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (2.089 forks)batch import
Code QualityH1 2020Help WantedMinordonotreap

Métricas do repositório

Stars
 (10.371 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Pesquise no código fonte todas as ocorrências de 'embedding size' e 'embeddingsize'. Elas aparecem nos agentes seq2seq, transformer e memnn. Use o mecanismo existente 'upgrade opt' (provavelmente em options.py ou similar) para criar um alias dos nomes antigos inconsistentes para um único nome unificado. Certifique se de que quaisquer arquivos de configuração existentes ainda funcionem mapeando os nomes antigos. Consulte os comentários da issue para obter orientações adicionais sobre o nome unificado preferido.
Pilha de tecnologia
pythonpytorch
Domain
developer experience
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
PythonParlAI codebase familiarityunderstanding of option naming conventions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65