facebookresearch/fairseq

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

Open

#3219 aperta il 7 feb 2021

Vedi su GitHub
 (2 commenti) (7 reazioni) (0 assegnatari)Python (6224 fork)batch import
enhancementhelp wantedstale

Metriche repository

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

Descrizione

🚀 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

Guida contributor

Direzione di ricerca
Esamina l'integrazione di fairseqlm per determinare se flashight può essere reso opzionale. Identifica i percorsi di codice che richiedono flashlight e proponi un refactoring per consentire l'uso di fairseqlm LM senza flashlight. Considera l'uso di import condizionali o un'architettura a plugin.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythonfairseq
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40