Skip to content

Conversation

@Stonepia
Copy link

@Stonepia Stonepia commented Nov 12, 2025

This PR makes test_float8_tensor.py device agnostic and supports XPU.

Dependencies

This PR dependends on following PR.

If 166056 not merged, 57 tests will fail with the error message:

Now _scaled_mm only supports per-tensor scaling for CPU backend.

Test Status

After the dependencies in the above are merged, the tests will be 428 passed and 28 skipped. All the skipped tests are due to the fbgemm check _is_fbgemm_gpu_genai_available().

# Skipped Lists
test_bmm: 2 tests
test_expected_gpu_kernel_fbgemm: 1 test.
test_fp8_conv_variants : 16 tests.
test_fp8_conv_skip_quant: 8 tests.
test_moe_weight_reshape_ops: 1 test.

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 12, 2025

🔗 Helpful Links

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

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

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 Nov 12, 2025
@Stonepia Stonepia changed the title [xpu] Make tests in test_float8_tensor.py device agnostic [xpu] [Test] Make tests in test_float8_tensor.py device agnostic Nov 12, 2025
@Stonepia
Copy link
Author

@pytorchbot label "xpu"

@pytorch-bot pytorch-bot bot added the xpu Intel XPU related features label Nov 12, 2025
@Stonepia Stonepia changed the title [xpu] [Test] Make tests in test_float8_tensor.py device agnostic [XPU] [Test] Make tests in test_float8_tensor.py device agnostic Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. xpu Intel XPU related features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant