kaldi-asr/kaldi

Fix registering of CuAllocatorOptions

Open

#2929 aperta il 21 dic 2018

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)Shell (5359 fork)batch import
bughelp wantedstale

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema riguarda la correzione della registrazione di CuAllocatorOptions. Il codice attuale registra opzioni globali ma non chiama SetOptions(). La soluzione consiste nel rimuovere la variabile globale e utilizzare un'istanza locale di CuAllocatorOptions in ogni binario che registra le opzioni, quindi chiamare SetOptions(). Il codice rilevante si trova probabilmente nel sorgente di Kaldi dove è definito CuAllocatorOptions. Controllare i commenti del maintainer @desh2608 per ulteriori indicazioni. Non esistono PR collegati.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60