Replies: 13 comments 1 reply
-
We don't (still) have a simple on/off toggle. Gotta use lists as true/false currently. I don't think I can make a basic widget toggle as that would be a front end extension. The rest seem doable besides downstream on/off. I haven't figured out how to do that without comfy throwing an error where other nodes expect a input. Error won't break anything, but not pretty either. Haha |
Beta Was this translation helpful? Give feedback.
-
For the widget, I did this: "Input": ("INT", {"default": 1, "min": 1, "max": 2}), for the input and ComfyUI is able to toggle between widget and input. I made one after you did because I had an old version of your WAS suite and didn't realize you made one aleady. The WAS Suite is an amazing tool and you've done so much with it! I'm really impressed with all the nodes you've made. |
Beta Was this translation helpful? Give feedback.
-
Oh sorry, here's how I made my node if you want to reference it: class ComfyRoll_InputImages:
|
Beta Was this translation helpful? Give feedback.
-
Hi, that sounds awesome! RockOfFire was helping me with switch nodes before we saw your latest update. He also has some code for a Load LoRA node incorporating a switch . I'll ask him to send it. Suzie |
Beta Was this translation helpful? Give feedback.
-
I prefer to have a number input for logic, because the base use is logic with dynamic workflows. Something causes the logic boolean to be either 1, or 0, and switches all the inputs. |
Beta Was this translation helpful? Give feedback.
-
Also added a none to lora: |
Beta Was this translation helpful? Give feedback.
-
That makes sense and I really like your implementation of the Load Lora node!! That's really cool!! :D |
Beta Was this translation helpful? Give feedback.
-
Yes that will be perfect, thank you! I'll update my templates with the new nodes when they come out. |
Beta Was this translation helpful? Give feedback.
-
I pushed the update awhile ago so should be able to pull. |
Beta Was this translation helpful? Give feedback.
-
@Suzie1 I've added a Image Power Noise node: |
Beta Was this translation helpful? Give feedback.
-
That looks so cool!! :D Those Image Power Noise nodes are amazing!! |
Beta Was this translation helpful? Give feedback.
-
Yes, thank you, these will be perfect! |
Beta Was this translation helpful? Give feedback.
-
Hello - I think it would be neat to have a node that takes a list and text and tests to see if the first includes the second, outputting a boolean number that can be used by things like WAS_VAE_Input_Switch - combined with the name_string return from the checkpoint loader, these can be used to e.g. build a whitelist of checkpoints that can use their own baked VAE. I have actually written something along these lines, but I'm not exactly sure how to use git to make a pull request. If it's useful though, it would be something like this - class WAS_Text_List_Contains:
def __init__(self):
pass
@classmethod
def INPUT_TYPES(cls):
return {
"required": {
"list": ("LIST", {"forceInput": True}),
"text": ("STRING", {"forceInput": True}),
},
}
RETURN_TYPES = ("NUMBER",)
FUNCTION = "list_contains"
CATEGORY = "WAS Suite/Text"
def list_contains(self, list, text):
if text in list:
return (1,)
else:
return (0,)
Thank you for your hard work on these extensions to Comfy. They have let me port through a lot of my existing workflows in ways that will give me significant room for expanding them in the future. (My apologies in advance if this node already exists and I just can't see it for some reason.) |
Beta Was this translation helpful? Give feedback.
-
Hi, Fantastic custom nodes! Thank you so much for all your hard work.
Here is my wish list for new custom nodes:
Suzie
https://civitai.com/user/Akatsuzi/
Beta Was this translation helpful? Give feedback.
All reactions