'nn' requirement missing

harvardnlp / seq2seq-attn

Sequence-to-sequence model with LSTM encoder/decoders and attention

http://nlp.seas.harvard.edu/code

MIT License

1.26k stars 278 forks source link

Closed ofirpress closed 8 years ago

ofirpress commented 8 years ago

The package 'nn' is a dependency for 'cunn' but it does not appear on the list in the README. Should probably be added to the list. Thanks

yoonkim commented 8 years ago

thanks! i've updated the readme to reflect this