WASasquatch / was-node-suite-comfyui

An extensive node suite for ComfyUI with over 210 new nodes
MIT License
1.15k stars 170 forks source link

Number Counter node should use a Logic Boolean Primitive input #375

Open andreszs opened 5 months ago

andreszs commented 5 months ago

Some of the nodes from this plugin seem to be written by different developers, because nodes like the Number Counter do not accept a Logic Boolean Primitive as an input for a simple true/false value, and instead expect a number:

image

If it's too late to fix this (because it could render existing workflows corrupted) an optional boolean input could be added to make it more logical for new workflows, allowing to connect the Boolean Primitive to the boolean input.

ALso, the Number Counter should allow converting the reset_bool value to widget to avoid this additional Logic Boolean node altogether.