Skip to content

Conversation

vkuzo
Copy link
Contributor

@vkuzo vkuzo commented Sep 2, 2025

Summary:

Undoes part of #2908 to make the message about missing .so files be a debug print instead of a warning. Reason: this always happens for builds without executorch ops.

Keeps the version mismatch log as a warning.

Test Plan:

Make this change locally in an install of torchao on an H100, verify warning no longer prints.

Reviewers:

Subscribers:

Tasks:

Tags:

Summary:

Undoes part of #2908 to make
the message about missing `.so` files be a debug print instead of a
warning.  Reason: this always happens for builds without executorch ops.

Keeps the version mismatch log as a warning.

Test Plan:

Make this change locally in an install of torchao on an H100, verify
warning no longer prints.

Reviewers:

Subscribers:

Tasks:

Tags:
Copy link

pytorch-bot bot commented Sep 2, 2025

🔗 Helpful Links

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

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, 12 Pending

As of commit 5acbc29 with merge base 1bb14f8 (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 Sep 2, 2025
@vkuzo vkuzo added the topic: bug fix Use this tag for PRs that fix bugs label Sep 2, 2025
Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

@vkuzo vkuzo merged commit f5a1d08 into main Sep 2, 2025
15 of 20 checks passed
vkuzo added a commit that referenced this pull request Sep 2, 2025
Summary:

Undoes part of #2908 to make
the message about missing `.so` files be a debug print instead of a
warning.  Reason: this always happens for builds without executorch ops.

Keeps the version mismatch log as a warning.

Test Plan:

Make this change locally in an install of torchao on an H100, verify
warning no longer prints.

Reviewers:

Subscribers:

Tasks:

Tags:
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. 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