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

Vanishing gradient notebook #93

Closed
wants to merge 1 commit into from

Conversation

karanchahal
Copy link
Contributor

Poornima and I have compared an LSTM and RNN and visualized the gradients with respect to the input. We see that the gradients for the RNN are much smaller compared to the LSTM.

We are able to train MNIST for a large input sequence with an LSTM and failed to do so with an RNN.

Hope this is useful. If we need to make any chances, please let us know !

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@Atcold
Copy link
Owner

Atcold commented Mar 3, 2020

Hmm... Let's discuss on the Slack channel before submitting new notebooks.

Anyhow, can you please get this to play with the existing notebooks on RNN and LSTM?
I'm referring to https://github.com/Atcold/pytorch-Deep-Learning/blob/master/08-seq_classification.ipynb, which I will be teaching tomorrow.
It would be helpful not having to go through another notebook in class, since it adds overhead and people will get confused quickly.

@Atcold
Copy link
Owner

Atcold commented Mar 3, 2020

I can try to help out merging this tomorrow.
Perhaps could you drop by my office?

@karanchahal
Copy link
Contributor Author

Sure, I'll drop by your office today.

@Atcold
Copy link
Owner

Atcold commented Mar 13, 2020

Any update on this, @karanchahal?

@karanchahal
Copy link
Contributor Author

@Atcold no, I have not made any progress on this as yet.

@Atcold
Copy link
Owner

Atcold commented Mar 16, 2020

Can you give it a try, when you get a chance?
Thaaaanks!

@Atcold
Copy link
Owner

Atcold commented Mar 30, 2020

Closing for the moment.

@Atcold Atcold closed this Mar 30, 2020
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

Successfully merging this pull request may close these issues.

None yet

2 participants