rgthree / rgthree-comfy

Making ComfyUI more comfortable!
MIT License
980 stars 76 forks source link

Consideration - Fast Actions Button - Is the 'action' button necessary? #33

Closed Trobinou closed 11 months ago

Trobinou commented 11 months ago

Is this button really necessary to confirm the choices? Indeed, I noticed that when the workflow loads, I have to start by pressing this button to validate the displayed choices. If I don't do that, the actions don't proceed as displayed. Furthermore, I often forget (my fault) to confirm my choices because, for me, it is logical that when I see 'enable,' for example, the action should be triggered without the need to validate it.

So my question is: could we do without this button?

rgthree commented 11 months ago

Hmm.. there's only one actual "button" on the Fast Actions Button node and that's what actually runs the actions. Is that what you mean by "validate" ?

So, taking the following example, it has one button ("Action!" and three combos to select which actions to run). fast_actions_button

Perhaps I'm misunderstanding, but the button surely needs to exist to click it :)

Trobinou commented 11 months ago

The choice of the term 'validate' was not appropriate, but indeed, it is about clicking the 'Action!' button to execute the selected actions. What I meant is, would it not be possible to choose the desired option, for example, for the 'seed' in your example, between 'Fixed-increment-decrement-randomize,' and have the choice executed without pressing the 'Action!' button?

aegis72 commented 11 months ago

I think the intention of this node might be as a macro…ie, one button that sets the workflow in a specific state. In my workflow AegisFlow, I’d connect a huge host of nodes that set various parameters and enabled some nodes while muting others…say, a single action button that eliminates all nodes from executing except the most basic setup for testing and experimentation.

Trobinou commented 11 months ago

You're right, especially as the pack is rich enough in nodes to do as I wish: [Mute / Bypass Relay] + [Fast Muter] or [Fast Bypasser] gives me complete satisfaction 👌 I'm closing this request as it seems to me that there's no reason for it.

aegis72 commented 11 months ago

EDIT: Confused this with another post :)