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

Memory management for unrolled graphs #19

Closed
emjotde opened this issue Sep 29, 2016 · 0 comments
Closed

Memory management for unrolled graphs #19

emjotde opened this issue Sep 29, 2016 · 0 comments
Assignees

Comments

@emjotde
Copy link
Member

emjotde commented Sep 29, 2016

Graphs that are unrolled for different batch size when they contain RNNs should share the entire memory in order to avoid allocation. Parameters should be protected, the rest could be overwritten.

It probably makes sense to change the Tensor class to not hold it's own memory, rather it should be a view on a segment of the memory the graph holds. This should be solved together with issue #9 .

@emjotde emjotde self-assigned this Sep 29, 2016
@emjotde emjotde closed this as completed Jun 4, 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

1 participant