Random seed support

[romeobuilderotti]

Finally we get reproducible workflows!

Changes:

• add seed field to generators
• change DPRandomGenerator to inherit from DPGeneratorNode
• Autorefresh is disabled by default since seed makes the result fully deterministic when arguments are the same (unless some underlying wildcard files change
• add DPMultilineString node that passes string as is without applying dynamicPrompts to it
• change PROMPT input time to STRING so that the text widget can be converted into input (intended for use with DPMultilineString for example)

Caveats:

• I'm new to modding ComfyUI so could mess up in the code
• DPJinja has seed input but ignores it at the moment
• I don't understand what is special about PROMPT and what unforeseen consequences it could have when I changed it to STRING
• I didn't update any docs/examples (and to lazy too do it, so maybe someone else can handle that)

As an offtopic, I see "view prompt" item in your to do list. There is a working implementation in Custom Scripts plugin but it involves some JS hacks

[Robzz]

I stumbled on your PR while looking for a way to get reproducible workflows using this extension and gave it a try. It does seem reproducible, but it also seems to break wildcards in the random prompts node (tested by dumping the generated prompt to the console using a node from the WAS node suite). Anyway, thank you for making this PR, this would be very nice to have ❤️

[romeobuilderotti]

@Robzz oh this is because of the change of DPRandomGenerator to inherit from DPGeneratorNode instead of DPAbstractSamplerNode as it doesn't initialize the wildcard_manager. TBH I don't exactly know why only random and combinatorial nodes had support for wildcards and not others, maybe wildcard support should be moved to a base class so that all nodes support it?

By the way, I'm currently working on wildcard pipes in a separate branch of my repo, so maybe common code for loading wildcards both from files and the pipe can be moved there.

Not sure whether other people think think that this approach to wildcards is useful. I use it for smaller item lists to split my large prompts into pieces while keeping everything inside the workflow.

[adieyal]

Hi @romeobuilderotti

Thanks for the PR. I'm looking at it now, but I don't understand why you removed the WildcardManager from the Random Prompt generator.

[romeobuilderotti]

@adieyal I made DPRandomGenerator inherit from DPGeneratorNode which is a better fit for random generation rather than DPAbstractSamplerNode which deals with deterministic generation. At least this is how I understood the need for 2 different base classes.

Probably in addition to that WildcardManager should be extracted to a separate file so that both DPAbstractSamplerNode and DPGeneratorNode support it. What is the reason why jinja, feeling_luckly and magicprompt didn't support wildcards? I didn't use file based WildcardManager so I lack experience with it