facebookresearch/fairseq

can we skip flashlight when using fairseqlm LM for decoding for wave2vec?

Open

#3.219 aberto em 7 de fev. de 2021

Ver no GitHub
 (2 comments) (7 reactions) (0 assignees)Python (6.224 forks)batch import
enhancementhelp wantedstale

Métricas do repositório

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

Description

🚀 Feature Request

When using fairseqlm LM, flashlight LM looks like just a wrapper. Can we remove the dependency to flashlight when fairseqlm is being used? It's so much a nightmare to build flashlight python binding.

Motivation

It's so much a nightmare to build flashlight python binding.

Pitch

Alternatives

Additional context

Guia do colaborador

Direção de pesquisa
Investigue a integração do fairseqlm para determinar se o flashlight pode ser tornado opcional. Identifique os caminhos de código que exigem o flashlight e proponha uma refatoração para permitir o uso do fairseqlm LM sem o flashlight. Considere usar importações condicionais ou uma arquitetura de plugin.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Funcionalidade
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.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonfairseq
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40