clab/dynet

"empty" RNNStates should be rigged to return zero vectors on .s(), .h(), etc

Open

#1322 aperta il 24 mar 2018

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (701 fork)github user discovery
good first issueminor bug

Metriche repository

Star
 (3433 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

having .s() return nothing on an RNN's initial state is very confusing (at least to me), as there must be some kind of value from which the RNN starts and not revealing it just hides what's going on (making things like automaton-extraction very difficult).

to my understanding, RNNStates with empty .s() are effectively treated as having zero-vectors in .add_input() computations (the matrix multiplication with the missing c/h vectors is simply skipped). It would be nice if .s(), .h(), .output(), etc reflected this.

Guida contributor

Direzione di ricerca
Esamina come RNNStates gestisce .s() e .h() vuoti nel codice sorgente; assicurati che restituiscano vettori zero quando sono vuoti.
Tech stack
cpp
Dominio
machine learning
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70