search

WASasquatch / was-node-suite-comfyui

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

Conflicted Nodes in the Manager #305

Open junaid3d opened 9 months ago

junaid3d commented 9 months ago

Installed comfyui today and needed WAS nodes for a workflow but it lists these conflicting nodes

Conflicted Nodes: Image Save [ymc-node-suite-comfyui], Save Text File [ymc-node-suite-comfyui]

WASasquatch commented 9 months ago

Dirty fix would be renaming the nodes in the NODE_CLASS_MAPPINGS dictionary. But I won't be renaming my nodes on account of some other custom nodes at this time. The WAS nodes need to rebuild with class names and a NODES_DISPLAY_NAME_MAPPINGS dictionary added, which would break everyones workflows since the release of this suite back when ComfyUI started.

It would make sense for that custom node to instead standardize with using class names for class mapping and display names as I am sure it has a much smaller footprint in the community.