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.

Sign up for GitHub

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

Jump to bottom

fix: update pytorch nightly to fix memory leak #255

Merged
merged 1 commit into from
Jan 20, 2023
Merged

fix: update pytorch nightly to fix memory leak #255

merged 1 commit into from
Jan 20, 2023

Conversation

pommedeterresautee
Copy link
Member

With current version of PyTorch there is a memory leak that can be reproduced if we follow the code below:
pytorch/torchdynamo#1955 (comment)

Updating PyTorch fixes the issue.

test pass

> pytest

================================================================================================== 2859 passed in 6454.06s (1:47:34) ===================================================================================================

@pommedeterresautee pommedeterresautee added the bug Something isn't working label Jan 19, 2023
@pommedeterresautee pommedeterresautee self-assigned this Jan 19, 2023
@github-actions github-actions bot added the fix hurrah, bug fixed! label Jan 19, 2023
@pommedeterresautee pommedeterresautee merged commit 306aa3a into main Jan 20, 2023
@pommedeterresautee pommedeterresautee deleted the fix/memory_leak_dynamo branch January 20, 2023 08:52
@pommedeterresautee pommedeterresautee mentioned this pull request Jan 20, 2023
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaetansnl gaetansnl gaetansnl approved these changes

Assignees

@pommedeterresautee pommedeterresautee

Labels
bug Something isn't working fix hurrah, bug fixed!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pommedeterresautee @gaetansnl