Skip to content

Revert "[rocm] scaled_grouped_mm support gfx942 fp8 data type"#3713

Merged
danielvegamyhre merged 1 commit into
mainfrom
revert-3540-dev
Jan 23, 2026
Merged

Revert "[rocm] scaled_grouped_mm support gfx942 fp8 data type"#3713
danielvegamyhre merged 1 commit into
mainfrom
revert-3540-dev

Conversation

@danielvegamyhre

@danielvegamyhre danielvegamyhre commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Reverts #3540

Unfortunately we need to revert this as it seems to have broken MoE training as some callsites haven't been properly updated to correctly handle the new param. Starting next week we'll support emulated mode to run these tests in CI, but in the meantime, we need to fix and run the tests locally before relanding @xiaobochen-amd

the tests are in this dir: https://github.com/pytorch/ao/tree/main/test/prototype/moe_training

the TP tests aren't expected to pass yet, but everything else should.

@pytorch-bot

pytorch-bot Bot commented Jan 23, 2026

Copy link
Copy Markdown

🔗 Helpful Links

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

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

✅ No Failures

As of commit d7f0b50 with merge base 81d7755 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla 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 Jan 23, 2026
@pytorch-bot

pytorch-bot Bot commented Jan 23, 2026

Copy link
Copy Markdown

Warning: Unknown label ciflow/rocm-mi300.
Currently recognized labels are

  • ciflow/benchmark
  • ciflow/tutorials
  • ciflow/rocm
  • ciflow/4xh100
  • ciflow/xpu

Please add the new label to .github/pytorch-probot.yml

@danielvegamyhre danielvegamyhre added the topic: bug fix Use this tag for PRs that fix bugs label Jan 23, 2026
@danielvegamyhre danielvegamyhre merged commit 1472872 into main Jan 23, 2026
19 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td ciflow/rocm-mi300 CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. device: rocm topic: bug fix Use this tag for PRs that fix bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants