Gourieff / comfyui-reactor-node

Fast and Simple Face Swap Extension Node for ComfyUI
GNU General Public License v3.0
1.64k stars 163 forks source link

Reactor doesn't respect the extra_model_paths.yaml (extra search path) #423

Open scofano opened 2 months ago

scofano commented 2 months ago

First, confirm

What happened?

When setting an external folder for the models (I create many portable backups), reactor forces download of the /facerestore_models /facedetection folders and doesn't recognize the /insightface model folder.

Steps to reproduce the problem

Your workflow on D:\ComfyUI_windows_portable\ComfyUI create a file called extra_model_paths.yaml and place new folder settings. Mine, as an example, is:

config for comfyui

your base path should be either an existing comfy install or a central folder where you store all of your models, loras, etc.

comfyui:

base_path: path/to/comfyui/

checkpoints: models/checkpoints/

clip: models/clip/

clip_vision: models/clip_vision/

configs: models/configs/

controlnet: models/controlnet/

embeddings: models/embeddings/

loras: models/loras/

upscale_models: models/upscale_models/

vae: models/vae/

other_ui: base_path: D:/ checkpoints: models/checkpoints/ clip: models/clip/ clip_interrogator: models/clip_interrogator/ clip_vision: models/clip_vision/ configs: models/configs/ controlnet: models/controlnet/ diffusers: models/diffusers/ diffusion_models: models/diffusion_models/ embeddings: models/embeddings/ gligen: models/gligen/ hypernetworks: models/hypernetworks/ LLM: models/LLM/ llm_gguf: models/llm_gguf/ loras: models/loras/ photomaker: models/photomaker/ style_models: models/style_models/ unet: models/unet/ upscale_models: models/upscale_models/ vae: models/vae/ vae_approx: models/vae_approx/ animatediff_models: models/animatediff_models animatediff_motion_lora: models/animatediff_motion_lora CogVideo: models/CogVideo facedetection: models/facedetection facerestore_models: models/facerestore_models insightface: models/insightface mmdets: models/mmdets onnx: models/onnx reactor: models/reactor sams: models/sams ultralytics: models/ultralytics xlabs: models/xlabs

Sysinfo

Win 11, rtx4070

Relevant console log

No log

Additional information

No response

ClothingAI commented 1 month ago

found a way to do it?

scofano commented 1 month ago

found a way to do it?

Nope and I found out that other custom nodes also do not respect the extra_model_path, so I'm using a mix of alternative path models and models on default location. Meh... @ClothingAI

ClothingAI commented 1 month ago

Some work some don't yeah .. it becomes complicated when youhave multiple local comfy installations..