mozilla/DeepSpeech

Change Alphabet::EncodeSingle and Alphabet::DecodeSingle to use an outparam and return an error code instead of calling `abort`

Open

#3175 aperta il 22 lug 2020

Vedi su GitHub
 (14 commenti) (0 reazioni) (0 assegnatari)C++ (4093 fork)batch import
buggood first bughelp wanted

Metriche repository

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

Descrizione

I run into the error "Invalid label 0". I assume it is from the function Alphabet::DecodeSingle() in the file native_client/alphabet.cc.

I don't know why I am running into this error and it is not important for now. What is important is the fact that the code calls abort() just after printing this error.

I don't know your guidelines. But I can tell that aborting the whole process, including the python interpreter which might be interactive, is a terrible way of handling errors, especially in such a trivial function, and I call it a bug.

Guida contributor

Direzione di ricerca
Modifica Alphabet::EncodeSingle e Alphabet::DecodeSingle in native client/alphabet.cc per utilizzare un parametro di output e restituire un codice di errore invece di chiamare abort.
Tech stack
cpp
Dominio
machine learningbackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60