Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PyTorch 2.5 to regression test #1168

Merged
merged 14 commits into from
Oct 25, 2024
Merged

Add PyTorch 2.5 to regression test #1168

merged 14 commits into from
Oct 25, 2024

Conversation

leslie-fang-intel
Copy link
Collaborator

Replace #1100

Copy link

pytorch-bot bot commented Oct 25, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 16b6aa7 with merge base eb1fb3a (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-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 25, 2024
@leslie-fang-intel
Copy link
Collaborator Author

cc @gau-nernst @msaroufim @jerryzh168

@msaroufim
Copy link
Member

msaroufim commented Oct 25, 2024

I think this is mergeable since we pinned nightly PyTorch. Mind just rebasing to main so we can see the sparse failures go away

@msaroufim msaroufim merged commit 2370758 into main Oct 25, 2024
29 of 30 checks passed
@msaroufim msaroufim deleted the leslie/torch2.5 branch October 25, 2024 03:22
@@ -104,7 +105,13 @@
def _int8wo_api(mod):
if TORCH_VERSION_AT_LEAST_2_4:
quantize_(mod, int8_weight_only(), set_inductor_config=False)
if not TORCH_VERSION_AT_LEAST_2_5:
if (
Copy link
Contributor

Choose a reason for hiding this comment

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

wow condition is becoming pretty complicated..

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yean, ensure as much as test combination has been covered.

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants