ComfyUI
ComfyUI copied to clipboard
comfyanonymous
%width% and %height% wildcards for filepath
Allows users to set the filename prefix of the SaveImage node to something like this : PR/%width%x%height%/ComfyUI
I don't have any opinion on the %var% syntax, I know it's commonly used so that's what I went with.
Yes, I miss that feature, but I think it should be more generic, maybe something node based, to allow any variable, people will ask for a lot of other variables.
I don't have a better solution right now, but we can discuss about it, maybe others can help us to find the best approach for this problem.
Yes, I miss that feature, but I think it should be more generic, maybe something node based, to allow any variable, people will ask for a lot of other variables.
I don't have a better solution right now, but we can discuss about it, maybe others can help us to find the best approach for this problem.
I agree.
A lot of this is probably covered by the fact we can convert widget to input now, tho. I did not test it but I think you can convert the filename_prefix to an input and build the string you want with nodes. This can be done frontend side. A "date" node maybe ?
The case covered by this PR is something that can't be done from the frontend.
Yes, the width and height of the image are a special case because it would be a pain to do it in the frontend, anything else should be done in the frontend.
Yes, I miss that feature, but I think it should be more generic, maybe something node based, to allow any variable, people will ask for a lot of other variables. I don't have a better solution right now, but we can discuss about it, maybe others can help us to find the best approach for this problem.
I agree.
A lot of this is probably covered by the fact we can convert widget to input now, tho. I did not test it but I think you can convert the
filename_prefixto an input and build the string you want with nodes. This can be done frontend side. A "date" node maybe ?The case covered by this PR is something that can't be done from the frontend.
True, now with the widget to input we just need some nodes to work with primitive types, like concatenate string, convert number to string, and things like that.