Skip to content
This repository was archived by the owner on Nov 22, 2022. It is now read-only.

Set no_grad context explicitly #1016

Closed
wants to merge 1 commit into from

Conversation

ArmenAg
Copy link
Contributor

@ArmenAg ArmenAg commented Oct 1, 2019

Summary: Multiple people have seen that GPU memory constantly increases during testing with SequenceGenerator. This diff explicitly insures that no graph is built.

Differential Revision: D17606381

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 1, 2019
ArmenAg added a commit to ArmenAg/pytext that referenced this pull request Oct 4, 2019
Summary:
Pull Request resolved: facebookresearch#1016

Multiple people have seen that GPU memory constantly increases during testing with SequenceGenerator. This diff explicitly insures that no graph is built.

Differential Revision: D17606381

fbshipit-source-id: b0dcaf3e54244635e99f96a9696dcfa1cb8f9520
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D17606381

Summary:
Pull Request resolved: facebookresearch#1016

Multiple people have seen that GPU memory constantly increases during testing with SequenceGenerator. This diff explicitly insures that no graph is built.

Differential Revision: D17606381

fbshipit-source-id: 4bdedb4939699449b51d0320776bcd3f1ae98374
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D17606381

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f82830d.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants