-
Notifications
You must be signed in to change notification settings - Fork 63
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
Sageattention2 fails to build on torch nightly 2.7.0 (8th-26th-27th-28th feb), because ambiguous symbol from torch dynamo (Windows) #101
Comments
can it work on ealier stable release of pytorch like torch 2.5.1? |
It works fine on torch 2.5.1 or 2.6.0. But for the future stable 2.7.0, not sure how It will behave. Could also report it on the pytorch repo. |
@Panchovix It's weird. People have successfully build SageAttention with torch 2.7 on RTX5090. What GPU do you use? |
Tried today again (after getting a RTX 5090) and still fails on windows nightly. Using torch 2.6.0 from https://huggingface.co/w-e-w/torch-2.6.0-cu128.nv/tree/main works. |
Fails to build on 2.7 nightly as well. Same error, "error C2872: 'std': ambiguous symbol" |
pytorch/pytorch#144707 is probably the problematic change. Commenting out these lines (Around line 1097 on my nightly checkout) in your torch in python site-packages:
Did allow sageattention to compile and run. Obviously if you had some code that did want to do gradients over strings/text you'd need to fix that properly 😂 (and upstream pytorch should fix this, it's nothing to do with sageattention specifically) |
Many thanks, it was that! Commented on the PR |
I got this same error when I tried to build a few days back but tried again with the latest nightly with |
Hi there, thanks for the great work.
I did update torch to latest nightly of 8th Feb, but got
Mostly to catch it now.
Using nightly of 1 week ago (2025-02-01) works fine. Probably some related dynamo changes affected this.
Thanks in advance!
The text was updated successfully, but these errors were encountered: