Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LSTM implementation #23

Open
mees opened this issue Feb 1, 2019 · 1 comment
Open

LSTM implementation #23

mees opened this issue Feb 1, 2019 · 1 comment

Comments

@mees
Copy link

mees commented Feb 1, 2019

Hi,
I wanted to ask about the LSTM implementation as it looks a bit different to the original implementation, for example it is actually not necessary to use an tf.nn.dynamic_rnn for variable sized sequences, as the inputs are already padded to have the same length, right?

@InnerPeace-Wu
Copy link
Owner

Yes, the input is padded as the same length. I did not give much attention on the implementation, i just found out the one worked for me, so there may be more suitable way to do the job.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants