kaldi-asr/kaldi

"bc" is not linux native , you should add some check

Open

#2 954 ouverte le 31 déc. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Shell (5 359 forks)batch import
enhancementhelp wantedstale

Métriques du dépôt

Stars
 (15 392 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

low priority issue , it is not a real issue to debug this error , and install bc afterwards , but nevertheless , this is not a native linux command . you should add some warning in , maybe in run.sh for all the diarizations recipe , or in the script itself.

https://github.com/kaldi-asr/kaldi/blob/3e77220b3297831c3a370169580b880461897682/egs/callhome_diarization/v1/diarization/extract_ivectors.sh#L95

Guide contributeur

Direction de recherche
Ajoutez une vérification dans le script pour confirmer si 'bc' est installé ; sinon, affichez un avertissement et installez le éventuellement. Voir la ligne référencée dans extract ivectors.sh.
Stack technique
shell
Domaine
devopsbackend
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
Shell scripting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80