mozilla/DeepSpeech

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

Ouverte

#3 175 ouverte le 22 juil. 2020

Voir sur GitHubGuide contributeur
 (14 commentaires) (0 réaction) (0 personne assignée)C++ (4 093 forks)batch import
buggood first bughelp wanted

Métriques du dépôt

Stars
 (26 755 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Modifiez Alphabet::EncodeSingle et Alphabet::DecodeSingle dans native client/alphabet.cc pour utiliser un paramètre de sortie et renvoyer un code d'erreur au lieu d'appeler abort.
Stack technique
cpp
Domaine
machine learningbackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60