[PR #942] [MERGED] feat: mapping clip and checkpoint to model path #1613

New issue

Closed

opened 2026-10-27 21:33:06 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/ComfyUI-Manager/pull/942
Author: @allmelgr
Created: 8/9/2024
Status: ✅ Merged
Merged: 8/9/2024
Merged by: @ltdrdata

Base: main ← Head: main

📝 Commits (1)

84c1983 feat: mapping clip and checkpoint to model path

📊 Changes

1 file changed (+4 additions, -0 deletions)

View changed files

📝 glob/manager_server.py (+4 -0)

📄 Description

Hi Ltdrdata,

I seen some change in model-list.json:
the type "checkpoints" was changed to "checkpoint", but in the glob.manager_server.get_model_dir without this mapping
and comfyanonymous/clip_l is default path, clip is also without mapping

And I seen the defualt path is etc, not in ComfyUI/models/etc. maybe use os.path.join(folder_paths.models_dir, "etc") is better?

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/ComfyUI-Manager/pull/942 **Author:** [@allmelgr](https://github.com/allmelgr) **Created:** 8/9/2024 **Status:** ✅ Merged **Merged:** 8/9/2024 **Merged by:** [@ltdrdata](https://github.com/ltdrdata) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`84c1983`](https://github.com/Comfy-Org/ComfyUI-Manager/commit/84c19839c43a1df9881a6d25af61661f7aa43785) feat: mapping clip and checkpoint to model path ### 📊 Changes **1 file changed** (+4 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `glob/manager_server.py` (+4 -0) </details> ### 📄 Description Hi Ltdrdata, I seen some change in model-list.json: the type "checkpoints" was changed to "checkpoint", but in the glob.manager_server.get_model_dir without this mapping and comfyanonymous/clip_l is default path, clip is also without mapping And I seen the defualt path is **etc**, not in ComfyUI/models/etc. maybe use `os.path.join(folder_paths.models_dir, "etc")` is better? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

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