Skip to content

[reland][ROCm] use dataclass for fnuz type setting #1150

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.

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

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

jeffdaily
Copy link
Contributor

Reland of #1142. Revert the revert #1148. Then check for torch.cuda.is_available() before trying to call torch.cuda.get_device_properties(0).

Copy link

pytorch-bot bot commented Oct 23, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/1150

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ You can merge normally! (1 Unrelated Failure)

As of commit ac575ba with merge base d252612 (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 23, 2024
@jeffdaily
Copy link
Contributor Author

@msaroufim how do we test whether we're breaking things this time around?

@msaroufim
Copy link
Member

We can trigger the failed GitHub action on your branch. AFK now but will handle merging this

@jeffdaily jeffdaily changed the title [reland][ROcm] use dataclass for fnuz type setting [reland][ROCm] use dataclass for fnuz type setting Oct 23, 2024
Copy link

pytorch-bot bot commented Oct 23, 2024

No ciflow labels are configured for this repo.
For information on how to enable CIFlow bot see this wiki

@msaroufim
Copy link
Member

Ok this worked in my test branch

@msaroufim msaroufim self-requested a review October 23, 2024 20:26
@msaroufim
Copy link
Member

Sparse test failures are unrelated

@msaroufim msaroufim merged commit 75d0693 into pytorch:main Oct 23, 2024
95 of 97 checks passed
yanbing-j pushed a commit to yanbing-j/ao that referenced this pull request Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/rocm CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: rocm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants