Skip to content
New issue

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

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

Already on GitHub? # to your account

Possible to Implement bi-LSTM? #1

Open
thombrey-vinil opened this issue Aug 22, 2018 · 1 comment
Open

Possible to Implement bi-LSTM? #1

thombrey-vinil opened this issue Aug 22, 2018 · 1 comment

Comments

@thombrey-vinil
Copy link

Is it possible to implement a bidirectional LSTM just by changing parameter 'bidirectional = True' in the code?

Please let me know if this works.

@Janinanu
Copy link
Owner

Janinanu commented Nov 6, 2018

I think it should work in general. But you might need to adjust the way you access and slice the dimensions, because the output of a bidirectional LSTM will have a different dimension than the output of a unidirectional LSTM, I suppose. See the PyTorch documentation for details. But I never tried it...
Also, using a bidirectional LSTM might lead to even more overfitting.

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

No branches or pull requests

2 participants