Closed aidangomez closed 7 years ago
@aidangomez I would like to help you out! :)
@yash14123 Awesome! We'll see what Google has to say about any already present implementations.
We'd love to have it in the models repo if anyone writes it: https://github.com/tensorflow/models!
I'm working on it
Just opened a pull request
@aidangomez Can you link to the pull request?
I see. Let me know if u need any help.
发件人: Aidan Gomez notifications@github.com 发送时间: 2016年6月13日 23:45 收件人: tensorflow/tensorflow 抄送: elvinpoon; Manual 主题: Re: [tensorflow/tensorflow] Implementation of CW-RNN? (#1280)
Here: tensorflow/models#198https://github.com/tensorflow/models/pull/198
― You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/1280#issuecomment-225740698, or mute the threadhttps://github.com/notifications/unsubscribe/ASuxNtpeByyBt7-1Ba7aV2XMa2u33XO6ks5qLeuMgaJpZM4HiMj-.
@girving The PR got updated recently and has a unit test that suggests everything is working as expected. Probably getting near what's acceptable for merger, do you mind having a look?
@aidangomez On leave, so someone else will have to review.
Closing. The PR is still open. I am not sure why it did not get resolved and the discussion should continue there.
Has anyone considered implementing the CW-RNN architecture discussed here: http://arxiv.org/abs/1402.3511
The cited results seem extremely promising, and the structure is quite simple. If it's not in the pipeline or already present, I may try implementing it myself.