Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect nickname badge of conflicted nodes such as comfyui_controlnet_aux #500

Open
Layer-norm opened this issue Mar 18, 2024 · 4 comments

Comments

@Layer-norm
Copy link

image
Most nodes from comfyui_controlnet_aux have been labled as ComfyUI-Inference-Core-Nodes since two week ago, due to the nodes conflicts.

As the ComfyUI community grows, nodes conflicts would become more and more common. People need to know which nodes they actually use. So is there any good way to distinguish these? I want the comfyui_controlnet_aux nickname badge back.

@ltdrdata
Copy link
Owner

This issue is already recognized. While some improvements can be made within ComfyUI-Manager, achieving a perfect resolution is technically impossible.
The most appropriate solution is for node developers to uniquely assign node names through prefixes or postfixes to avoid collisions.

@ltdrdata
Copy link
Owner

ltdrdata commented Mar 19, 2024

For now, the original cnet aux has been preemptively reserved for those node names.

@Fannovel16
Copy link

Fannovel16 commented May 29, 2024

@ltdrdata I suggest either hiding or removing Inference-Core from ComfyUI-Manager as it forks cnet aux

@ltdrdata
Copy link
Owner

@ltdrdata I suggest either hiding or removing Inference-Core from ComfyUI-Manager as it forks cnet aux

I'll handle it later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants