FR: pipeKSamplerAdvanced

TinyTerra / ComfyUI_tinyterraNodes

A selection of nodes for Stable Diffusion ComfyUI

GNU General Public License v3.0

421 stars 45 forks source link

Closed DrJKL closed 1 year ago

DrJKL commented 1 year ago

Mainly looking for start_at_step, end_at_step, return_with_leftover_noise to be able to refactor some existing workflows.

TinyTerra commented 1 year ago

Added pipeKSamplerAdvanced, let me know if it's not working as expected :)

DrJKL commented 1 year ago

Mostly :-D

2 notes:

I forgot (didn't notice when posting) add_noise as one of the options in the built-in Advanced
I also didn't see that the pipeKSampler and now the pipeKSamplerAdvanced have their own seed widgets by default instead of taking the seed from the pipe and exposing optional_seed as an input.

Thanks for the quick response!

TinyTerra commented 1 year ago

Updated it, hopefully that's better now

DrJKL commented 1 year ago

I think it's not picking up the seed from the pipe, maybe because it's seed there, but noise_seed in the Sampler?

DrJKL commented 1 year ago

I'll try to put 2 repros together

DrJKL commented 1 year ago

workflow

TinyTerra commented 1 year ago

Ah of course. Sorry about that. Latest push should fix it:

DrJKL commented 1 year ago

Looks good, thanks!