kaldi-asr/kaldi

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

Open

#2954 aperta il 31 dic 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Shell (5359 fork)batch import
enhancementhelp wantedstale

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Aggiungi un controllo nello script per verificare se 'bc' è installato; in caso contrario, stampa un avviso e, opzionalmente, installalo. Vedi la riga in extract ivectors.sh.
Tech stack
shell
Dominio
devopsbackend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
Shell scripting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80