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
Improve docs of links/connection/lstm #3104
Improve docs of links/connection/lstm #3104
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the comment.
chainer/links/connection/lstm.py
Outdated
There are several ways to make a LSTM link. | ||
|
||
Let a two-dimensional input array :math:`x`, a cell state array | ||
:math:`h`, and the output array of the previous step :math:`h` be: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The cell state array and the output array of the previous step are not used the following part, so I think the part which describes about these two variables should be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed it.
@keisuke-umezawa Could you rebase it to the current master? |
5fc96a6
to
6037984
Compare
@mitmul I rebased and followed review. |
Jenkins, test this please |
LGTM |
This is PR for improving docs of functions/links. Related issue: #2182
Modified functions/links: