-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document solution to.... UE for list-select widgets converted to inputs.... (Method to apply Anything to various inputs?) #69
Comments
Currently not possible. It's not quite like the reroute nodes, but it's not totally separate either. Anything that is a dropdown has a weird data type. Renaming this to remind me to think about it more.... |
I think this was the issue, right? #52 |
So, what's the solution? I'd like to have both sampler and scheduler set in one place. |
This issue is open. |
As a quick solution - maybe, for now, just add statically-typed |
UE_sampler_control_example.json I think I found another way to do it for now. CustomSampler, provide the sampler and sigmas via UE. |
It's nice... but it basically makes any other sampling nodes incompatible with UE. Like, effeciency pack, impact pack or similar. |
I wonder if they could change their widget/input to be compatible with the built-in Sampler/Sigma providers |
Ooops... my bad. I didn't know there already exists a ... However, |
Oddly enough, it does work with the node from image-saver. |
I think the KSampler expects a |
Great. I have a solution for now, with no need to create any custom nodes. And since there is already a node for that, I don't think you need to add your own doing the same thing. Though, you might want to add this workflow suggestion to readme. |
good move with the sampler/scheduler selectors. I'll start incorporating them in my workflows |
+1. The difference between KSamplerSelect and Sampler Selector from image-saver seems to be the type: |
For reference, there is now a way to control sampler/scheduler globally in the inspire pack: ltdrdata/ComfyUI-Inspire-Pack@fb7ebc7#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 |
Or do with #91 |
This is working for me too :) Please keep posting new findings. Good job! |
Closing this - but I've referenced it in the docs. Thanks to those who worked out solutions! Regarding working with third party samplers - I'd be encouraging their authors to make them work with the built in Sampler and Sigma types... |
There is also a workaround, with the |
maybe an enhancement suggestion...as i couldn't find any prior issue directly addressing this (though i see something similar often repeated )... say i want to set scheduler to be global.
I go to my ksamplers, set scheduler to be "input"
then when i use the everything/anywhere node.... i'd like that to affect all the schedulers that have input
is this possible? or is it part of that issue where you mentioned recently that reroute nodes are difficult?
The text was updated successfully, but these errors were encountered: