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

NO MATTER WHAT I DO I CANT INSTALL ANY PULID CUSTOM NODE...even this one #10

Open
Maki9009 opened this issue Jan 22, 2025 · 13 comments
Open

Comments

@Maki9009
Copy link

yeah the title says it all, i keep getting errors for numpy, facelibx.. iv research i used claude and still it just wont work.

@lldacing
Copy link
Owner

lldacing commented Jan 23, 2025

You need to provide full error logs.

@lycos2019
Copy link

Traceback (most recent call last):
File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\nodes.py", line 2106, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 999, in exec_module
File "", line 488, in call_with_frames_removed
File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\custom_nodes\comfyui_pulid_flux_ll_init
.py", line 1, in
from .pulidflux import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\custom_nodes\comfyui_pulid_flux_ll\pulidflux.py", line 11, in
from insightface.app import FaceAnalysis
ModuleNotFoundError: No module named 'insightface'

how do i solve this? seems it conflicts with a-person-mask-generator

@lldacing
Copy link
Owner

Traceback (most recent call last): File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\nodes.py", line 2106, in load_custom_node module_spec.loader.exec_module(module) File "", line 999, in exec_module File "", line 488, in call_with_frames_removed File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\custom_nodes\comfyui_pulid_flux_ll__init_.py", line 1, in from .pulidflux import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS File "C:\Users\miggu\Desktop\AI\ComfyUI\ComfyUI\custom_nodes\comfyui_pulid_flux_ll\pulidflux.py", line 11, in from insightface.app import FaceAnalysis ModuleNotFoundError: No module named 'insightface'

how do i solve this? seems it conflicts with a-person-mask-generator

Did you installed the plugin dependencies
pip install -r requirements.txt

@DragonDiffusionbyBoyo
Copy link

DragonDiffusionbyBoyo commented Jan 25, 2025

If the nodes do not load it is because filterpy is not installing and the pip installer tries to get the wrong one.
For the portable version do this command and it will get the right version:

python.exe -m pip install git+https://github.com/rodjjo/filterpy.git

Then run the install requirements this will now install all other requirements correctly without falling over.
All Pulid nodes will now work.

I suggest the owner of the repo put the path I have put above in the requirements.txt so that this loads and works for everyone.

@lldacing
Copy link
Owner

If the nodes do not load it is because filterpy is not installing and the pip installer tries to get the wrong one.
For the portable version do this command and it will get the right version:

python.exe -m pip install git+https://github.com/rodjjo/filterpy.git

Then run the install requirements this will now install all other requirements correctly without falling over.
All Pulid nodes will now work.

I suggest the owner of the repo put the path I have put above in the requirements.txt so that this loads and works for everyone.

I'm not sure this is a problem. I just test the command pip install facexlib in the new environment and didn't encounter any errors. The filterpy is installed correctly.

@DragonDiffusionbyBoyo
Copy link

Hello,
I saw posts in this repo and other Pulid threads on other repos regarding nodes failing to install.
I recreated the error on two pcs with several different versions of comfy installations. Most errored and this was the fix that resolved the issue on all of my machines and installations.
I just thought it useful for people who are having installation issues. (like the ones I recreated)
Regards

@robertJene
Copy link

robertJene commented Jan 26, 2025

SOLUTION

Image

I have VENV version of comfy for flux stuff. But this fix should work for portable because you are doing it from command line.

I had the error in the manager:
(IMPORT FAILED) ComfyUI_PuLID_Flux_II

I fixed it using DragonDiffusionbyBoyo's instructions.

Here's the steps:

cd custom_nodes
cd ComfyUI_PuLID_Flux_ll
python.exe -m pip install git+https://github.com/rodjjo/filterpy.git
pip install -r requirements.txt

If the nodes do not load it is because filterpy is not installing and the pip installer tries to get the wrong one. For the portable version do this command and it will get the right version:

python.exe -m pip install git+https://github.com/rodjjo/filterpy.git

Then run the install requirements this will now install all other requirements correctly without falling over. All Pulid nodes will now work.

I suggest the owner of the repo put the path I have put above in the requirements.txt so that this loads and works for everyone.

@lldacing
Copy link
Owner

SOLUTION

Image

I have VENV version of comfy for flux stuff. But this fix should work for portable because you are doing it from command line.

I had the error in the manager:
(IMPORT FAILED) ComfyUI_PuLID_Flux_II

I fixed it using DragonDiffusionbyBoyo's instructions.

Here's the steps:

cd custom_nodes
cd ComfyUI_PuLID_Flux_ll
python.exe -m pip install git+https://github.com/rodjjo/filterpy.git
pip install -r requirements.txt

If the nodes do not load it is because filterpy is not installing and the pip installer tries to get the wrong one. For the portable version do this command and it will get the right version:

python.exe -m pip install git+https://github.com/rodjjo/filterpy.git

Then run the install requirements this will now install all other requirements correctly without falling over. All Pulid nodes will now work.

I suggest the owner of the repo put the path I have put above in the requirements.txt so that this loads and works for everyone.

I use conda. It is no problem to execute commands in the order in the README. But if you don't configurate sys env for python, It should be full path of python.exe for portable.I will verify the installation of the manager later.

@robertJene
Copy link

also you need to write better instructions on your REPO.

Like, for example, what models to download and what folders they go in

Image

This is not sufficient instructions.

Image

@robertJene
Copy link

That's it. I've spent over an hour trying to setup your repo in my Comfy, and I have to give up and move on with my life.

Image

@lldacing
Copy link
Owner

also you need to write better instructions on your REPO.

Like, for example, what models to download and what folders they go in

Image

This is not sufficient instructions.

Image

Yes, it is planned. You can download the model according to the introduction in the original project first.

@DragonDiffusionbyBoyo
Copy link

I might have fixed that error with flux face loader. I am just testing the code now. Will post once testing is completed

@lldacing
Copy link
Owner

lldacing commented Jan 26, 2025

That's it. I've spent over an hour trying to setup your repo in my Comfy, and I have to give up and move on with my life.

Image

please see #11 last comment

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

No branches or pull requests

5 participants