-
Notifications
You must be signed in to change notification settings - Fork 103
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
Adding comfyui attention mode #2
Comments
same problem here. |
The issue isn't adding them, they just don't work with the model, even the original code only works with flash_attn. Sdpa is already an option, but it didn't produce movement on my tests so far. Both flash and sage can be installed on Windows, it's not super simple but very much possible. Sage is also a lot faster so it can be worth the effort, depending on your GPU. |
Ok thanks for the answer do you know where there's a tutorial for install it on windows because pip install never works for both |
sdpa working for me but as you said no movement just pixelated movements |
It's more complicated than that, Purz made this guide a while ago that should still apply: https://purz.notion.site/Get-Windows-Triton-working-for-Mochi-6a0c055e21c84cfba7f1dd628e624e97 |
Here's a prebuilt triton package from this huggingface repo: |
Did that and now getting this error. |
is weird cause after i finally figured how to make Triton works for Mochi (wich is actually working now)
sdpa gives me pixelated results too
|
the same error msg. |
i would add at least the word "Sage" in the main title of this post or something like that |
The varlen version for sage was added later, so you probably need to update sageattention itself. |
i can confirm what Kijai sayd. Sdpa produce really subtle movements. |
ok pasting here in case other users looking for same solution: this solved. Now works. Thanks Kijai🤍 |
If you are on Windows and using the Portable version of ComfyUI then there is a critical step explained on the triton-windows github under: That was the last piece of the puzzle for me. |
Might've finally figured it out, at least I can now use sdpa: AnimateDiff_00002.63.mp4 |
how was that possible? |
* TextImage Encoder now a seperate node.
Hi flash_attn and sageattn doesn't works on my windows 10 (24go de vram ) I tried everything for install it without sucess can you add others attention mode like the comfyui one ? thanks !
The text was updated successfully, but these errors were encountered: