[GH-ISSUE #105] Duplicate model and 2 face_restore directories #61

New issue

Closed

opened 2026-10-27 21:12:33 +00:00 by bsakaguchi · 2 comments

bsakaguchi commented

2026-10-27 21:12:33 +00:00

Owner

Originally created by @coreyryanhanson on GitHub (Sep 12, 2023).
Original GitHub issue: https://github.com/Comfy-Org/ComfyUI-Manager/issues/105

I noticed in the model list there is one section for face_restore that defaults to a face_restore directory:
github.com/ltdrdata/ComfyUI-Manager@0d50544287/model-list.json (L93C1-L122C7)

But also at the same time there appears to be a newer addition where models are saved to the face_restore_models directory:
github.com/ltdrdata/ComfyUI-Manager@0d50544287/model-list.json (L1137C5-L1176C7)

Existing in both is GFPGANv1.4 (but implying a later release in the hyperlink on the second one)

Is there a reason to keep both sections unconsolidated or is the first face_restore output path planned to be deprecated?

Originally created by @coreyryanhanson on GitHub (Sep 12, 2023). Original GitHub issue: https://github.com/Comfy-Org/ComfyUI-Manager/issues/105 I noticed in the model list there is one section for face_restore that defaults to a face_restore directory: https://github.com/ltdrdata/ComfyUI-Manager/blob/0d50544287da108dc7992a04925ee97df319f087/model-list.json#L93C1-L122C7 But also at the same time there appears to be a newer addition where models are saved to the face_restore_models directory: https://github.com/ltdrdata/ComfyUI-Manager/blob/0d50544287da108dc7992a04925ee97df319f087/model-list.json#L1137C5-L1176C7 Existing in both is GFPGANv1.4 (but implying a later release in the hyperlink on the second one) Is there a reason to keep both sections unconsolidated or is the first face_restore output path planned to be deprecated?

bsakaguchi closed this issue

2026-10-27 21:12:33 +00:00

bsakaguchi commented

2026-10-27 21:12:33 +00:00

Author

Owner

@ltdrdata commented on GitHub (Sep 12, 2023):

Unfortunately, the problem lies in the fact that custom nodes require different paths.
face_restore is the path used in the MTB node, while facerestore_models is the path used in the later added Facerestore CF (Code Former) node.

Due to this reason, there are several duplicate items available for download for each path.

@ltdrdata commented on GitHub (Sep 12, 2023): Unfortunately, the problem lies in the fact that custom nodes require different paths. `face_restore` is the path used in the `MTB` node, while `facerestore_models` is the path used in the later added `Facerestore CF (Code Former)` node. Due to this reason, there are several duplicate items available for download for each path.

bsakaguchi commented

2026-10-27 21:12:33 +00:00

Author

Owner

@coreyryanhanson commented on GitHub (Sep 12, 2023):

Makes sense. Hopefully in the future one of the paths will win out

@coreyryanhanson commented on GitHub (Sep 12, 2023): Makes sense. Hopefully in the future one of the paths will win out