Skip to content

Conversation

@jeffdaily
Copy link
Collaborator

Revert "Skips some complex tests on ROCm (#42759)". This reverts commit 55b1706.

Use new cuda_to_hip_mappings.py from #43004.

Fixes #42383 (comment)

CC @sunway513

Revert "Skips some complex tests on ROCm (pytorch#42759)"
This reverts commit 55b1706.

Use new cuda_to_hip_mappings.py from pytorch#43004.
@jeffdaily jeffdaily added the module: rocm AMD GPU support for Pytorch label Aug 27, 2020
@jeffdaily jeffdaily requested review from anjali411 and ngimel August 27, 2020 21:14
@codecov
Copy link

codecov bot commented Aug 28, 2020

Codecov Report

Merging #43744 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #43744   +/-   ##
=======================================
  Coverage   69.34%   69.34%           
=======================================
  Files         378      378           
  Lines       46680    46680           
=======================================
+ Hits        32369    32370    +1     
+ Misses      14311    14310    -1     
Impacted Files Coverage Δ
torch/utils/hipify/cuda_to_hip_mappings.py 100.00% <ø> (ø)
torch/testing/_internal/expecttest.py 78.57% <0.00%> (+1.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdc3e23...e5406a6. Read the comment docs.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

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

Copy link
Collaborator

@ngimel ngimel left a comment

Choose a reason for hiding this comment

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

Looks good, waiting for ci.

Copy link
Contributor

@anjali411 anjali411 left a comment

Choose a reason for hiding this comment

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

LGTM thank you :D

@facebook-github-bot
Copy link
Contributor

@ngimel merged this pull request in 8fb7c50.

@jithunnair-amd jithunnair-amd deleted the rocm_complex_blas branch September 25, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: rocm AMD GPU support for Pytorch open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants