Skip to content

Conversation

@Xia-Weiwen
Copy link
Collaborator

This reverts commit 30082cb.

The commit was reverted because of #3175. However, the issue was not caused by this commit but a bug in setup.py. And the issue will be fixed via #3187.

So, this PR re-apply the commit that was reverted.

Original PR: #3100 and #3052

Test plan

pytest -sv test/test_ops.py -k test_float8_linear_cpu

@pytorch-bot pytorch-bot bot added the ci-no-td label Oct 29, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 29, 2025

🔗 Helpful Links

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

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:

✅ No Failures

As of commit 460208e with merge base 9266734 (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 Oct 29, 2025
@Xia-Weiwen Xia-Weiwen added the topic: not user facing Use this tag if you don't want this PR to show up in release notes label Oct 29, 2025
@Xia-Weiwen
Copy link
Collaborator Author

CC @mingfeima for review. Thanks.

@Xia-Weiwen Xia-Weiwen marked this pull request as ready for review October 30, 2025 00:33
@Xia-Weiwen
Copy link
Collaborator Author

Hi @vkuzo Would you mind if we add this back? Removing this does not resolve #3175 after all. Thanks.

@Xia-Weiwen
Copy link
Collaborator Author

Hi @vkuzo @jerryzh168 Do you think we can land this now? Thanks.

@Xia-Weiwen
Copy link
Collaborator Author

Hi @vkuzo @jerryzh168 Would you mind if we land this now? It's not the cause of building errors on Windows. Thanks.

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

Labels

ci-no-td CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing Use this tag if you don't want this PR to show up in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant