KEMBAR78
empty caching allocator before test_avg_pool2d large subtest by jeffdaily · Pull Request #63528 · pytorch/pytorch · GitHub
Skip to content

Conversation

@jeffdaily
Copy link
Collaborator

Otherwise, unrecoverable OOM occurs on MI25. Fixes broken ROCm CI test1.

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Aug 18, 2021

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 21af453 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@codecov
Copy link

codecov bot commented Aug 19, 2021

Codecov Report

Merging #63528 (21af453) into master (f596aa8) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #63528   +/-   ##
=======================================
  Coverage   75.53%   75.53%           
=======================================
  Files        2119     2119           
  Lines      212492   212525   +33     
=======================================
+ Hits       160500   160533   +33     
  Misses      51992    51992           

@jeffdaily jeffdaily marked this pull request as ready for review August 19, 2021 15:27
@jeffdaily jeffdaily requested a review from walterddr August 19, 2021 15:27
…test

Otherwise, unrecoverable OOM occurs on MI25.
@jeffdaily jeffdaily force-pushed the rocm_test_nn_empty_cache branch from 058f07f to 59b6741 Compare August 19, 2021 15:36
@jeffdaily jeffdaily requested a review from walterddr August 19, 2021 17:11
@mruberry mruberry added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Aug 19, 2021
@facebook-github-bot
Copy link
Contributor

@walterddr has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@walterddr merged this pull request in a8de0d8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants