search

giriss / comfy-image-saver

All the tools you need to save images with their generation metadata on ComfyUI. Compatible with Civitai & Prompthero geninfo auto-detection. Works with png, jpeg and webp.
MIT License
188 stars 32 forks source link

[BUG] Checkpoint Selector node breaking search width #5

Open Chubbly opened 1 year ago

Chubbly commented 1 year ago

The Checkpoint selector node is returning a type that is a list of every checkpoint a person has. If someone has many checkpoints, this stretches out the search box (the thing that pops up when you double click on the canvas) to larger than the the viewport width, basically making search unusable. The primitive version uses a COMBO type, I'm wondering if this could be used as a return type instead.

WinstonWoof commented 1 year ago

same issue

arkinson9 commented 1 year ago

same issue too

Chubbly commented 1 year ago

After investigating this as much as my (in)expertise will allow, the unfortunate conclusion is that there is no way to solve this with the current architecture of ComfyUI. The standard Load Checkpoint node expects a list checkpoints so the only way to fix this would be to change how the default Load Checkpoint node works.

For people wishing for a solution that still allows them to use the handy saving function from this repo, you can change the ckpt_name to an input widget and then double-click the input node to create a Combo primitive that can be passed in to the Save Image node from this repo. Or you can use the Load Checkpoint Text node from this repo https://github.com/lilly1987/ComfyUI_node_Lilly and pipe the value in, with the added benefit of using wildcards for checkpoint names. In either case, you will need to comment out class CheckpointSelector.

@giriss Unless you have spotted some other solution?