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
Implement MGU #1101
Implement MGU #1101
Conversation
Do you think we should explicitly name |
chainer/links/connection/mgu.py
Outdated
def set_state(self, h): | ||
assert isinstance(h, chainer.Variable) | ||
h_ = h | ||
if self.xp == numpy: |
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.
Use is
to compare modules.
We need to make a policy about names of RNN algorithms first. |
chainer/links/connection/mgu.py
Outdated
return h_new | ||
|
||
|
||
class StatefulMGU(MGU): |
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.
It have the same problem as one in SGD and DSGU in #1115. StatefulMGU
may not be the subclass of MGU
.
@unnonouno How about changing the name from |
As the naming rule of RNN modules is determined. We can merge this PR. Could you resolve the conflict? |
Resolved conflict and rewrote to use |
jenkins, test this please. |
test passed |
I implemented Minimal Gated Unit.
http://arxiv.org/abs/1603.09420