search

wallish77 / wlsh_nodes

WLSH ComfyUI Nodes
GNU General Public License v3.0
96 stars 13 forks source link

image save with file info error #17

Closed kakachiex2 closed 11 months ago

kakachiex2 commented 11 months ago

I'm getting the following errors wen traying to save image with prompt file info Screenshot 2023-11-15 151728

wallish77 commented 11 months ago

I can't tell from your screenshot for sure, but I'm wondering if it's because of the output path. If you leave that empty (so it defaults to the regular Comfy output) does it still give the error?

kakachiex2 commented 11 months ago

Give the same error removing the path I' really like your implementation

wallish77 commented 11 months ago

Ok, found the problem! I had misread this as the Image Save with Prompt/Info node and couldn't find any issues (in code or in tests). Just saw that it's the version that does the file and found the bug in the code. Newest commit should fix. Thanks for bringing this to my attention!

kakachiex2 commented 11 months ago

Nice thanks wallish

kakachiex2 commented 11 months ago

New error; know it save the image but not the text file Screenshot 2023-11-22 214741

wallish77 commented 11 months ago

Whoops sorry didn't actually PUSH the fix. Should be good in newest commit.

kakachiex2 commented 11 months ago

How can I save the image in a folder with the date do I have to add some parameter

kakachiex2 commented 11 months ago

Nitido pana work great thanks wallish