kaldi-asr/kaldi

Fix registering of CuAllocatorOptions

Ouverte

#2 929 ouverte le 21 déc. 2018

Voir sur GitHub
 (6 commentaires) (0 réaction) (1 personne assignée)Shell (5 359 forks)batch import
bughelp wantedstale

Métriques du dépôt

Stars
 (15 392 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Le problème concerne la correction de l'enregistrement de CuAllocatorOptions. Le code actuel enregistre des options globales mais n'appelle pas SetOptions(). La correction consiste à supprimer la variable globale et à utiliser une instance locale de CuAllocatorOptions dans chaque binaire qui enregistre des options, puis à appeler SetOptions(). Le code pertinent se trouve probablement dans la source de Kaldi où CuAllocatorOptions est défini. Consultez les commentaires du mainteneur @desh2608 pour des conseils supplémentaires. Aucune PR liée n'existe.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
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.
1-3 heures
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.
Plutôt claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60