kaldi-asr/kaldi

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

Aberta

#2.954 aberto em 31 de dez. de 2018

Ver no GitHub
 (3 comentários) (0 reação) (0 responsável)Shell (5.359 forks)batch import
enhancementhelp wantedstale

Métricas do repositório

Stars
 (15.392 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Adicione uma verificação no script para confirmar se 'bc' está instalado; caso contrário, exiba um aviso e, opcionalmente, instale o. Veja a linha referenciada no extract ivectors.sh.
Pilha de tecnologia
shell
Domain
devopsbackend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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
Shell scripting
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80