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

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

Open
3 tasks done
scofano opened this issue Sep 18, 2024 · 3 comments
Open
3 tasks done
Labels
bug Something isn't working new

Comments

@scofano
Copy link

scofano commented Sep 18, 2024

First, confirm

  • I have read the instruction carefully
  • I have searched the existing issues
  • I have updated the extension to the latest version

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

@scofano scofano added bug Something isn't working new labels Sep 18, 2024
@ClothingAI
Copy link

found a way to do it?

@scofano
Copy link
Author

scofano commented Oct 18, 2024

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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new
Projects
None yet
Development

No branches or pull requests

2 participants