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

Couldn't Achieve the Animation #494

Open
peterxcx opened this issue Oct 29, 2024 · 13 comments · May be fixed by #498
Open

Couldn't Achieve the Animation #494

peterxcx opened this issue Oct 29, 2024 · 13 comments · May be fixed by #498

Comments

@peterxcx
Copy link

Hello! I attempted to follow the same workflow, with the only variation being the checkpoint. However, I was unable to achieve the desired animation. All 16 images I obtained were not as anticipated; they appeared to be discontinuous.

At this point, I have no idea of this issue. Any assistance would be greatly appreciated. Thank you in advance!

Animatediff

@dariiboi
Copy link

same issue here, debug shows no errors. using most recent version of animatediff-evolved...

[32m[rgthree] Using rgthree's optimized recursive execution.�[0m [AnimateDiffEvo] - �[0;32mINFO�[0m - Sliding context window activated - latents passed in (20) greater than context_length 16. [AnimateDiffEvo] - �[0;32mINFO�[0m - Using motion module animatediffMotion_v15V2.ckpt:v2.

@Kosinkadink
Copy link
Owner

I don't see any obvious changes in ComfyUI that would cause the issue, so maybe some other node pack had some update. Please post more info here when you can!

@peterxcx
Copy link
Author

I don't see any obvious changes in ComfyUI that would cause the issue, so maybe some other node pack had some update. Please post more info here when you can!

I think I’ve found the problem, which is caused by ComfyUI itself. When I tried to update the entire ComfyUI from version 2024-05-28 to 2024-09-27, it worked well. Thanks for your reply!

@peterxcx
Copy link
Author

same issue here, debug shows no errors. using most recent version of animatediff-evolved...

[32m[rgthree] Using rgthree's optimized recursive execution.�[0m [AnimateDiffEvo] - �[0;32mINFO�[0m - Sliding context window activated - latents passed in (20) greater than context_length 16. [AnimateDiffEvo] - �[0;32mINFO�[0m - Using motion module animatediffMotion_v15V2.ckpt:v2.

If you have already updated AnimateDiff, you can try updating your ComfyUI (you could check your version). I solved the issue by updating the entire system(from 2024-05-28 to 2024-09-27).

@Kosinkadink
Copy link
Owner

Yeah you need to not be months outdated for it to work

@JRR948
Copy link

JRR948 commented Oct 30, 2024

I have same issue but using the latest ComfyUI and the AnimateDiff.
Tried same settings with peterxcx.

Only the difference is, I used hotshotxl instead of SD1.5 based animatediff model.

@Kosinkadink
Copy link
Owner

Please provide full list of custom nodes, AnimateDiffEvolved version, and ComfyUI version.

@JRR948
Copy link

JRR948 commented Oct 30, 2024

explorer_kGlV8Xccl8

This is all I installed, The ComfyUI version is latest git version.
comfyanonymous/ComfyUI@09fdb2b

@peterxcx
Copy link
Author

peterxcx commented Nov 5, 2024

explorer_kGlV8Xccl8

This is all I installed, The ComfyUI version is latest git version. comfyanonymous/ComfyUI@09fdb2b

Have you solved the problem yet? If not, could you please provide your results using both ComfyUI and the command line?

@JRR948
Copy link

JRR948 commented Nov 7, 2024

What command line btw?

image

This is result on ComfyUI

@Kosinkadink
Copy link
Owner

Your result is just as if AD wasn't on. I can't replicate it at all. There will be an update I'll be pushing in the next week or so that will launch alongside a new ComfyUI update, where a bunch of the AD injection will become native to ComfyUI that should hopefully eliminate all possible edge cases that could cause this.

@peterxcx
Copy link
Author

What command line btw?

image

This is result on ComfyUI

When you open your comfyUI, it will display a command prompt window, typically characterized by a black background, similar to this.

image

image

If we could see your code in your cmd, which may help us find out what actually happen in your ComfyUI.

@Kosinkadink
Copy link
Owner

Once my ComfyUI hooks PR gets merged (current ETA is around US Thanksgiving), I'll merge my ADE PR as well. Since model injection will be a native feature, there should be no way for this sort of issue to have any edge cases. #498

@Kosinkadink Kosinkadink linked a pull request Nov 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants