search

Danand / ComfyUI-ComfyCouple

Attention Couple made easier for ComfyUI.
GNU General Public License v3.0
30 stars 8 forks source link

Loving it, but the simplicity comes at a cost #2

Open HaydenReeve opened 8 months ago

HaydenReeve commented 8 months ago

Hello, 👋

Loving your work and glad to have you around.

I've noticed a small issue with the simplicity of this implementation though—you lose some control over small artefacts like character height.

By specifying exact masks you can be more rigorous with "I want this character to fill this area," but with a simple divide at positon x or y you generally end with an output where character heights or differences in posture are genericised. Not something you can really prompt around depending on the checkpoint.

Perhaps a way to allow for 'advanced overrides' might solve this issue. Or an 'advanced version' of the node in parallel which would give you the best of both worlds.

Either way, this is not a mission-critical, this is just an observation of the implementation's side effects. It's still a great node and your hard work is appreciated!

Danand commented 7 months ago

@HaydenReeve Hi! Glad you liked it!

I guess, the same issue is presented in the original node. Oh, I noticed it has been moved to archive recently, btw. So sad.

Maybe another approaches like sketching with ControlNet could do a trick.