-
Notifications
You must be signed in to change notification settings - Fork 768
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
Update attention.py #116
base: main
Are you sure you want to change the base?
Update attention.py #116
Conversation
Adding support for cards that aren't Ampere architecture
Thanks shirubei, I tried the code with my 8*2080ti. |
I got further with your branch but now getting OOM issues on my VRAM. Good try though |
Same here, for single 2080ti, i got OOM as well. Multiple GPUs not working. :( |
you can go with --frame_num 17 or even less to test the modification, such as 13 or 9. |
@splendiz @jimbojd72 python generate.py --task t2v-1.3B --size 832*480 --ckpt_dir |
Getting the same error with your new prompt. I don't know enough for now to debug on my side (first time trying a model on my Arch with 2080 Ti). My point here is that I should not be a blocker for merging if it doesn't work on my setup.
|
@jimbojd72 Thank you. |
Now I understand the link between frame_num and vram 🤦🏻 . Thanks for clarifying that for me. It did work afterward even though the video seems too short!
t2v-1.3B_832.480_1_1_Two_anthropomorphic_cats_in_comfy_boxing_gear_and__20250302_232400.mp4 |
I have tested this.It need frame_num under about 12 to make sure 2080ti 22G run smoothly.It cost 5min to generate a 832480 video |
single gpu may work, how about multi gpus? Anyone tested the code? |
Adding support for cards that aren't Ampere architecture