mozilla/DeepSpeech

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

Open

#3.175 aberto em 22 de jul. de 2020

Ver no GitHub
 (14 comments) (0 reactions) (0 assignees)C++ (4.093 forks)batch import
buggood first bughelp wanted

Métricas do repositório

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

Description

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.

Guia do colaborador

Direção de pesquisa
Modifique Alphabet::EncodeSingle e Alphabet::DecodeSingle em native client/alphabet.cc para usar um parâmetro de saída e retornar um código de erro em vez de chamar abort.
Pilha de tecnologia
cpp
Domain
machine learningbackend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60