search

SamKhoze / ComfyUI-DeepFuze

DeepFuze is a state-of-the-art deep learning tool that seamlessly integrates with ComfyUI to revolutionize facial transformations, lipsyncing, Face Swapping, Lipsync Translation, video generation, and voice cloning.
https://cogidigm.com/
Other
213 stars 13 forks source link

Your requirements.txt is too specific #7

Closed ManOrMonster closed 1 week ago

ManOrMonster commented 2 weeks ago

This node broke other nodes. I'm assuming it's because of the requirements.txt with very specific requirements.

SamKhoze commented 1 week ago

Hello @ManOrMonster,

can you let me know what are the dependencies or packages is conflicting?

ManOrMonster commented 1 week ago

Numpy was one. It broke ComfyUI-Mana-Nodes and I think at least one other custom node, but I don't remember which, and my log for yesterday is gone now. I was getting Numpy errors for sure.

Long story short, I ended up uninstalling this node then using "try fixing" from Manager on Mana Nodes, and it miraculously solved the issue.

kelvinyoung commented 1 week ago

Numpy was one. It broke ComfyUI-Mana-Nodes and I think at least one other custom node, but I don't remember which, and my log for yesterday is gone now. I was getting Numpy errors for sure.

Long story short, I ended up uninstalling this node then using "try fixing" from Manager on Mana Nodes, and it miraculously solved the issue.

Ya, when TTS requires numpy==1.22.0, meanwhile scipy requires 1.24.6

SamKhoze commented 1 week ago

Now you don't need to install TTS externally and we integrated it in with our current repo

ManOrMonster commented 1 week ago

The current requirements.txt has librosa, scipy, and tqdm listed twice each.

SamKhoze commented 1 week ago

Thanks for letting me know It is updated.