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

compatiblity patch for removed controlnet to device #59

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ltdrdata
Copy link
Contributor

Related commit:
comfyanonymous/ComfyUI@754597c

This patch solves this issue:
comfyanonymous/ComfyUI#5344 (comment)

@ntrouve-onera
Copy link

Tested, it works.
We need this rep to be made alive again, still the best nodes for tile on comfyui (Need flux support though, just outputing noise atm)

@JackDainzh
Copy link

Breaks every SDXL and probably non sdxl cnets. The typical missmatch in value types.The device handling is moved now into ControlNet class and called load_device.

fix: adaptive empty latent channel
modified: min tile size 256 -> 128
@ltdrdata
Copy link
Contributor Author

Breaks every SDXL and probably non sdxl cnets. The typical missmatch in value types.The device handling is moved now into ControlNet class and called load_device.

Updated. Try again.

@mmartial
Copy link

FYSA, I encountered this exact error, which brought me to this PR

I confirm having tested the patch (in order):

  • Applied it:
cd custom_nodes/ComfyUI_TiledKSampler
cp nodes.py{,.orig}
wget https://github.com/BlenderNeko/ComfyUI_TiledKSampler/pull/59.patch
patch -p1 < 59.patch
  • Used Restart from ComfyUI Manager Menu
  • Reload the browser page
  • Rerun failing workflow

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

Successfully merging this pull request may close these issues.

5 participants