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

Feature: Sparse Access Memory #4

Closed
ixaxaar opened this issue Oct 27, 2017 · 5 comments
Closed

Feature: Sparse Access Memory #4

ixaxaar opened this issue Oct 27, 2017 · 5 comments

Comments

@ixaxaar
Copy link
Owner

ixaxaar commented Oct 27, 2017

Implement sparse access memory for DNCs.
Related paper: https://arxiv.org/abs/1610.09027

@MaxwellRebo
Copy link
Contributor

Are you already diving into this, or do you need some help?

@ixaxaar
Copy link
Owner Author

ixaxaar commented Nov 10, 2017

Not looking at this at the moment. Trying to make DNCs work for language modeling & seq2seq.

@MaxwellRebo
Copy link
Contributor

I've done DNCs for seq2seq before. Does anything about the base implementation need to change, or is it just a matter of structuring the inputs and outputs?

@ixaxaar
Copy link
Owner Author

ixaxaar commented Nov 17, 2017

I actually wrote this repo because the other ones were not torch.nn-like. Making the APIs more like torch.nn makes it easier to replace nn.LSTM etc with DNC otherwise keeping implementations intact, potentially requiring one line of code change.

The ones I tried, I had to also write this piece of code for LMs to detach hidden states: https://gist.github.com/ixaxaar/02b65312eb1a840cc14e74d93b35178f
For seq2seq I took the openNMT repo and forked it here.

Other than that it was pretty much a few liners.

@ixaxaar ixaxaar mentioned this issue Nov 27, 2017
Merged
@ixaxaar
Copy link
Owner Author

ixaxaar commented Dec 17, 2017

Is implemented in #16 and #18 , though code review etc are pending.

@ixaxaar ixaxaar closed this as completed Dec 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants