kaldi-asr/kaldi

Fix registering of CuAllocatorOptions

Open

#2.929 aberto em 21 de dez. de 2018

Ver no GitHub
 (6 comments) (0 reactions) (1 assignee)Shell (5.359 forks)batch import
bughelp wantedstale

Métricas do repositório

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

Description

Currently it seems we register the global options but don't call the SetOptions() function. There is no need for the global options-class variable, we can use a local one in each binary that registers the options. @desh2608 can you please do this? Ask one of the other students if you don't know how.

Guia do colaborador

Direção de pesquisa
O problema é sobre corrigir o registro de CuAllocatorOptions. O código atual registra opções globais, mas não chama SetOptions(). A correção é remover a variável global e usar uma instância local de CuAllocatorOptions em cada binário que registra opções, então chamar SetOptions(). O código relevante provavelmente está na fonte do Kaldi onde CuAllocatorOptions é definido. Verifique os comentários do mantenedor @desh2608 para mais orientações. Não há PRs vinculados.
Pilha de tecnologia
cpp
Domain
backend
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.
1
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.
Principalmente claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60