Skip to content

Move Lynx ArcFace weights to ComfyUI models directory#1728

Open
mossmatrix wants to merge 2 commits into
kijai:mainfrom
mossmatrix:move-weights-to-comfy-models
Open

Move Lynx ArcFace weights to ComfyUI models directory#1728
mossmatrix wants to merge 2 commits into
kijai:mainfrom
mossmatrix:move-weights-to-comfy-models

Conversation

@mossmatrix

Copy link
Copy Markdown
Contributor

Changes the download location for the ArcFace recognition model weights from the custom node directory to the centralized ComfyUI models directory.

  • Register new 'face_analysis' model folder at ComfyUI/models/face_analysis
  • Update init_recognition_model() to accept and use model_rootpath parameter
  • Update FaceEncoderArcFace.init_encoder_model() to pass model root path
  • Update LynxEncodeFaceIP node to provide face_analysis directory path

Weights now download to: ComfyUI/models/face_analysis/facexlib/weights/
Instead of: custom_nodes/ComfyUI-WanVideoWrapper/lynx/face/facexlib/weights/

Benefits:

  • Centralized model storage following ComfyUI conventions
  • Easier model management and reuse
  • Models persist across node reinstalls

Changes the download location for the ArcFace recognition model weights
from the custom node directory to the centralized ComfyUI models directory.

- Register new 'face_analysis' model folder at ComfyUI/models/face_analysis
- Update init_recognition_model() to accept and use model_rootpath parameter
- Update FaceEncoderArcFace.init_encoder_model() to pass model root path
- Update LynxEncodeFaceIP node to provide face_analysis directory path

Weights now download to: ComfyUI/models/face_analysis/facexlib/weights/
Instead of: custom_nodes/ComfyUI-WanVideoWrapper/lynx/face/facexlib/weights/

Benefits:
- Centralized model storage following ComfyUI conventions
- Easier model management and reuse
- Models persist across node reinstalls
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 this pull request may close these issues.

1 participant