ComfyUI Search and replace strings in WAS Save Text File and Save Image nodes

[masslevel]

Maybe I haven't figured out the correct syntax, but is it possible to use the Search and replace strings of ComfyUI in WAS Save nodes? This way WAS custom tokens and widget information could be used to construct filenames.

Right now the Search and replace strings feature only seems to work in the default ComfyUI Save Image node.

I tried putting %node_name.widget_name% into the filename_prefix widget in the WAS Image Save Node and also the Text Add Tokens node. But the variables were not populated.

The search and replace strings feature is unofficially documented here: https://blenderneko.github.io/ComfyUI-docs/Interface/SaveFileFormatting/

[WASasquatch]

As far as I knew these were part of a custom_node by Data It. If they are in base ComfyUI it's a bug that they don't work on custom nodes.

[masslevel]

Thanks for your reply and thank you for your work, @WASasquatch! The WAS node suite helped me to build some really cool components for my workflow.

I see. I didn't know that. I thought because the Node name for S&R option is available in every native and custom node under the node's properties, it would be something custom nodes could hook into.

[WASasquatch]

It's front-end functionality, so there wouldn't even by anything for custom_nodes to do, it would be replaced by the front-end JavaScript handling the widgets. "widgets" don't exist on the back end. That's why I suggested it may be a bug.

[WinstonWoof]

May I turn this on its head a little then.

The image Below shows 4 WAS Nodes

If I wanted to pass the Checkpoint name to a text file output how would I do it?

[WASasquatch]

Convert text_b to a input and hook up the checkpoint name output from the loader to it.

…

On Thu, Aug 3, 2023, 10:41 AM WinstonWoof ***@***.***> wrote: May I turn this on its head a little then. The image Below shows 4 WAS Nodes If I wanted to pass the Checkpoint name to a text file output how would I do it? [image: image] <https://user-images.githubusercontent.com/128506111/258192271-8b393261-17a8-4101-a5d3-8079c46afe2f.png> — Reply to this email directly, view it on GitHub <#129 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIZEZIUU4WK33EENOJPGZ3XTPPERANCNFSM6AAAAAA2T4NYJI> . You are receiving this because you were mentioned.Message ID: <WASasquatch/was-node-suite-comfyui/repo-discussions/129/comments/6630454@ github.com>

[WinstonWoof]

will give it a whirl

[WinstonWoof]

Ok so that works but then it also shows the issue.

If I want to , for example, pull in the Sampler Name then because even your version of the Ksampler doesnt have the text outputs I cant connect it the same way.

IF your Nodes were able to use the built in S&R Functionality this would be far easier for the end user IMHO

[WASasquatch]

You would need to bring that up with ComfyUI. That isn't a my nodes issues. That's ComfyUI and using LiteGraph which is a mess.

…

On Sun, Aug 6, 2023, 3:39 PM WinstonWoof ***@***.***> wrote: Ok so that works but then it also shows the issue. If I want to , for example, pull in the Sampler Name then because even your version of the Ksampler doesnt have the text outputs I cant connect it the same way. *IF* your Nodes were able to use the built in S&R Functionality this would be far easier for the end user IMHO — Reply to this email directly, view it on GitHub <#129 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIZEZME7J52CYQFINLUE6TXUAMIJANCNFSM6AAAAAA2T4NYJI> . You are receiving this because you were mentioned.Message ID: <WASasquatch/was-node-suite-comfyui/repo-discussions/129/comments/6653043@ github.com>

[WinstonWoof]

so using you Nodes only how would I pass the sampler name & scheduler name to the .txt output please?

[WASasquatch]

Currently you wouldn't until ComfyUI fixes that and allows widget tokens to be used in custom node fields I guess. Even the primitive node is handled on the front end so not sure I could make a node that converts the combo value to a string. The only work around I can think of which just dawned on me before sending this, is to print the widget code in primitive under util. That in theory should print the value in it, which then could be used as the value of a WAS-NS token which can be printed in the WAS-NS save node.

…

On Sun, Aug 6, 2023, 11:01 PM WinstonWoof ***@***.***> wrote: so using you Nodes only how would I pass the sampler name & scheduler name to the .txt output please? — Reply to this email directly, view it on GitHub <#129 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIZEZIH3OWUNC4RILTG7NDXUCAEVANCNFSM6AAAAAA2T4NYJI> . You are receiving this because you were mentioned.Message ID: <WASasquatch/was-node-suite-comfyui/repo-discussions/129/comments/6654936@ github.com>

[WinstonWoof]

see I was thinking of it the other way.

Something similar to your Parse Tokens node that could Parse S&R terms and pipe them to a text output

[WASasquatch]

Problem with that, again, is it would be a custom node and thus S&R wouldn't work. I'd have the implement it myself, and I don't cover for bugs in mainline. They should just be fixed.

…

On Sun, Aug 6, 2023, 11:37 PM WinstonWoof ***@***.***> wrote: see I was thinking of it the other way. Something similar to your Parse Tokens node that could Parse S&R terms and pipe them to a text output — Reply to this email directly, view it on GitHub <#129 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIZEZLL7KMORFGIB7NNTA3XUCEMNANCNFSM6AAAAAA2T4NYJI> . You are receiving this because you were mentioned.Message ID: <WASasquatch/was-node-suite-comfyui/repo-discussions/129/comments/6655188@ github.com>