search

receyuki / comfyui-prompt-reader-node

The ultimate solution for managing image metadata and multi-tool compatibility. ComfyUI node version of the SD Prompt Reader
MIT License
289 stars 22 forks source link

SDParameterGenerator's compatibility with ComfyUI Browser's XYZ Plot #70

Open alessandroperilli opened 6 months ago

alessandroperilli commented 6 months ago

Summary

Hi @receyuki, long time we don't speak! I hope you are well.

I'm writing to see if you could double-check the compatibility between your SDParameterGenerator node and the SelectInputs node in the ComfyUI Browser custom node suite.

I opened an issue with the node author, but the message seems to be "it works with the default checkpoint loader node, so it's not my problem" :)

I want to be mindful of both yours and his/her time, so I'm double-checking on your side before going back to the other side.

Thank you for any time you could dedicate to this.

Basic Example

Please check https://github.com/talesofai/comfyui-browser/issues/33

Reference Issues.

No response

receyuki commented 6 months ago

Hi @alessandroperilli , I'm glad to see you here again! I just updated the standalone version to 1.3.5-beta.2 and now I'm starting to work on the issues in this repo. I will find out the reason ASAP and get back to you.

receyuki commented 6 months ago

You need to use slashes instead of backslashes in the XYZ Plot, because the ckpt_name in parameter generator uses forward slashes rather than backslashes. I also tested the default checkpoint loader, and it can only work properly when the slashes are used correctly.

alessandroperilli commented 6 months ago

I appreciate you checking this so quickly. I'm having another issue with that node that prevents me from testing the / solution. I hope the node answers and helps soon. Then, I'll let you know.

receyuki commented 6 months ago

No worries, just give me a shout when the node issue is sorted