Closed achyudh closed 5 years ago
@Impavidity can you please review this pull request?
One more suggestion, basically for the implementation of RNN model, we should provide length and use rnn_utils.pack_padded_sequence
. This is a more reasonable implementation (do not means this will get better results). If you have extra time, you could compare these two implementations.
A reference here https://github.com/castorini/Castor/blob/master/conv_rnn/model.py#L55
@Impavidity thanks for the tip. I added padding for packed sequences in #152 along with TensorboardX support. I didn't want to crowd this PR so made a new one. Also, I'll continue to push commits there so there is no need for a review right now.
Baseline LSTM
Implementation of a standard LSTM using PyTorch and Torchtext for text classification baseline measurements
Model Type
Quick Start
To run the model on Reuters dataset on static, just run the following from the Castor working directory.
Dataset
We experiment the model on the following datasets.
Settings
Adadelta is used for training.
TODO