search

WASasquatch / was-node-suite-comfyui

An extensive node suite for ComfyUI with over 210 new nodes
MIT License
1.15k stars 170 forks source link

When loading the graph, the following node types were not found: Seed, KSampler (WAS) #184

Open RonGeorge opened 1 year ago

RonGeorge commented 1 year ago

I think this is just a naming issue. If you load this image up in Comfy I get an error for 2 modules. SDVN6-RealXL-Detailface-57548091428964

KSampler (WAS) : but I can load KSampler (Legacy) and KSampler (Cycle) 9M0PiBu

. Legacy appears to have the same IOs, would there be a way that I can edit the name of it locally so these will all load without having to edit them one by one if I want to use them?

On a sidenote, you think I can do that with Seed as well?

RonGeorge commented 1 year ago

Just to add, the original image is from the SDVN Prompt kit, https://civitai.com/models/126425/sdvn-sdxlpromptkit

Oh, it seems that StableDiffusionVN is using that node in all of his stuff. https://civitai.com/images/1950310?modelVersionId=137456&prioritizedUserIds=1491543&period=AllTime&sort=Most+Reactions&limit=20

WASasquatch commented 1 year ago

Oh I didn't know it was relied upon anywhere. I can revert the name change.

RonGeorge commented 1 year ago

It is weird that it loads them by name instead of some unique number ID.

WASasquatch commented 1 year ago

It is weird that it loads them by name instead of some unique number ID.

Oh seriously. We already map the unique class name, to a text name... Then we map that text name to a "display name". Seems very convoluted when the class name is unique enough, and we have node IDs 🤷

WASasquatch commented 1 year ago

Should be reverted.