search

fofr / cog-comfyui

Run ComfyUI with an API
https://replicate.com/fofr/any-comfyui-workflow
MIT License
544 stars 124 forks source link

add-comfyui-tooling-nodes #50

Closed influgenai closed 7 months ago

influgenai commented 7 months ago

Hello! I would love to add the ComfyUi tooling Nodes to the repo. Thanks in advance!

ctscoville commented 7 months ago

@influgenai there's a specific save image node in that ComfyUI Tooling Nodes of type SendImageWebSocket. It would be nice to support the SendImageWebSocket node so that you don't need to write image files to disk and then read from them before returning a response. Have you tried that node as well and does it work already, or do you think we'd need to modify this PR to be able to handle image data directly in the outputs (instead of handling file names) when using that node?

fofr commented 7 months ago

Thank you for adding this 🙏

fofr commented 7 months ago

@ctscoville As you suspected, the Send Image (WebSocket) node does not work when using this directly – the model currently returns any files that are in the outputs directory, as this doesn't write any files, nothing gets returned.

A follow-up PR is needed to support Send Image (WebSocket).