Skip to content
This repository has been archived by the owner on Oct 17, 2021. It is now read-only.

Dispose intermediate RNN outputs to avoid running out of memory. #316

Merged
merged 4 commits into from
Sep 10, 2018

Conversation

adarob
Copy link
Member

@adarob adarob commented Sep 7, 2018

PERF


For repository owners only:

Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY

For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md


This change is Reviewable

@adarob
Copy link
Member Author

adarob commented Sep 7, 2018

FYI, tests now pass.

Copy link
Contributor

@caisq caisq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks a lot, @adarob !

@caisq caisq merged commit 79932e7 into tensorflow:master Sep 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants