Skip to content

Conversation

vkuzo
Copy link
Contributor

@vkuzo vkuzo commented Sep 2, 2025

Summary:

Fix error in #2908. The version string for PyTorch 2.8 reads "2.8.0...", so we need to compare >= 2.9 to properly gate out PyTorch 2.9.

Test Plan:

  1. make this change in a locally installed init file of torchao downloaded via pip
  2. install PyTorch 2.8.0
  3. import torchao, verify warning was not hit

Reviewers:

Subscribers:

Tasks:

Tags:

Summary:

Fix error in #2908. The version string for
PyTorch 2.8 reads "2.8.0...", so we need to compare `>= 2.9` to properly
gate out PyTorch 2.9.

Test Plan:

1. make this change in a locally installed __init__ file of torchao
   downloaded via pip
2. install PyTorch 2.8.0
3. import torchao, verify warning was not hit

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/2918

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

As of commit 94a83d8 with merge base ffabe80 (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
@vkuzo vkuzo merged commit 1bb14f8 into main Sep 2, 2025
18 of 20 checks passed
vkuzo added a commit that referenced this pull request Sep 2, 2025
Summary:

Fix error in #2908. The version string for
PyTorch 2.8 reads "2.8.0...", so we need to compare `>= 2.9` to properly
gate out PyTorch 2.9.

Test Plan:

1. make this change in a locally installed __init__ file of torchao
   downloaded via pip
2. install PyTorch 2.8.0
3. import torchao, verify warning was not hit

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