Solve the NaN issue on the CPU

tmbdev / clstm

A small C++ implementation of LSTM networks, focused on OCR.

Apache License 2.0

821 stars 224 forks source link

Solve the NaN issue on the CPU #67

Closed ASDen closed 6 years ago

ASDen commented 8 years ago

This PR solves the NaN issue reported here : https://github.com/tmbdev/clstm/issues/63 The problem is that for newly allocated batches inside a Sequence, memory had to be reset to zeros, otherwise NaNs may flow in

wanghaisheng commented 8 years ago

why still not merge ?

kba commented 6 years ago

Tested, works, thank you!