kaldi-asr/kaldi

Fix registering of CuAllocatorOptions

Open

#2.929 geöffnet am 21. Dez. 2018

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Shell (5.359 Forks)batch import
bughelp wantedstale

Repository-Metriken

Stars
 (15.392 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem betrifft die Korrektur der Registrierung von CuAllocatorOptions. Der aktuelle Code registriert globale Optionen, ruft aber SetOptions() nicht auf. Die Lösung besteht darin, die globale Variable zu entfernen und in jeder Binärdatei, die Optionen registriert, eine lokale CuAllocatorOptions Instanz zu verwenden und dann SetOptions() aufzurufen. Der relevante Code befindet sich wahrscheinlich in der Kaldi Quelle, wo CuAllocatorOptions definiert ist. Überprüfen Sie die Kommentare des Maintainers @desh2608 für weitere Anleitungen. Es gibt keine verknüpften PRs.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60