Skip to content

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