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

Incompatible with new RefCN functionality in ComfyUI-Advanced-ControlNet #20

Open
sjuxax opened this issue Apr 3, 2024 · 0 comments
Open

Comments

@sjuxax
Copy link

sjuxax commented Apr 3, 2024

Probably a "not your bug" bug, but opening here for awareness/tracking purposes, and in case there's something that can be done on this side to mitigate the issue. Kosinkadink wrote on Discord:

ref cn will not work with tiled diffusion at this time
the cn would need to have access to a vae during sampling to encode the input image per tile, and currently it prepares them beforehand. i'll eventually refactor code to make it compatible with existing tiling solutions, but for the time being i don't want to introduce potential memory usage issues without putting in the proper time to investigate

Should probably make a corresponding issue on ComfyUI-Advanced-ControlNet. I'll update here if I do that. See also Kosinkadink/ComfyUI-Advanced-ControlNet#96

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

1 participant