RPG-DiffusionMaster icon indicating copy to clipboard operation
RPG-DiffusionMaster copied to clipboard

Comfy Implementation

Open ShadoW-Shinigami opened this issue 1 year ago • 8 comments

Are there any plans for a comfyui node Implementation for the same?

ShadoW-Shinigami avatar Jan 23 '24 15:01 ShadoW-Shinigami

Yeah, comfy workflow is flexible. I will try to do that.

threefoldo avatar Jan 23 '24 22:01 threefoldo

second this! would LOVE to see this in comfyui

alenknight avatar Jan 24 '24 00:01 alenknight

Superb idea.

davizca avatar Jan 24 '24 00:01 davizca

Thanks for your valuable suggestions, we will try to do that and welcome collaborators

YangLing0818 avatar Jan 24 '24 00:01 YangLing0818

Yes this would be awesome !

julien-blanchon avatar Jan 24 '24 09:01 julien-blanchon

Just chiming in here to say, I too, would love to see this on ComfyUI. :)

Thanks for your hard work with RPG!

WASasquatch avatar Jan 26 '24 00:01 WASasquatch

Super excited to see this @threefoldo

peteromallet avatar Jan 26 '24 15:01 peteromallet

oh great news! Any ETA?

thibaudart avatar Jan 26 '24 21:01 thibaudart

+1 for supporting ComfyUI by @comfyanonymous

slavakurilyak avatar Jan 28 '24 08:01 slavakurilyak

Super excited to see this @threefoldo

Haven't maken any progress. The code is mixed with webui code, not sure how to seperate them.

threefoldo avatar Jan 28 '24 16:01 threefoldo

As a workaround if the code is too intertwined with the GUI, could we not create a wrapper node which calls the RPG python script passing in the arguments, and return the output image from it? Just as a temporary solution.

chrish-slingshot avatar Jan 29 '24 13:01 chrish-slingshot

As a workaround if the code is too intertwined with the GUI, could we not create a wrapper node which calls the RPG python script passing in the arguments, and return the output image from it? Just as a temporary solution.

Probably, yes, but ComfyUI wouldn't be aware of the resources that RPG use, so it could OOM people loading up ComfyUI models and RPG models.

WASasquatch avatar Jan 29 '24 16:01 WASasquatch

Sounded easier when I thought of it but I didnt think about the 23.5 GB VRAM you need on LLM single alone when region prompting in the areas... will be tough.

davizca avatar Jan 29 '24 17:01 davizca

Oh whoa. Yeah ... Hmm 🤔

On Mon, Jan 29, 2024, 9:34 AM davizca @.***> wrote:

Sounded easier when I thought of it but I didnt think about the 23.5 GB VRAM you need on LLM single alone when region prompting in the areas... will be tough.

— Reply to this email directly, view it on GitHub https://github.com/YangLing0818/RPG-DiffusionMaster/issues/1#issuecomment-1915235603, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIZEZODCKPEKWITFXHF3P3YQ7MSVAVCNFSM6AAAAABCHF43DGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGIZTKNRQGM . You are receiving this because you commented.Message ID: @.***>

WASasquatch avatar Jan 29 '24 20:01 WASasquatch

Sounded easier when I thought of it but I didnt think about the 23.5 GB VRAM you need on LLM single alone when region prompting in the areas... will be tough.

Yes, the runtime memory is about 15GB VRAM, not sure about the peak value. It's still a valid solution to treat this as a single node in comfyui.

threefoldo avatar Jan 30 '24 02:01 threefoldo

There's code for regional in Comfy in @ltdrdata's Inspire Pack that may be useful: https://github.com/ltdrdata/ComfyUI-Inspire-Pack

peteromallet avatar Jan 30 '24 11:01 peteromallet

This repo looks like a game changer. A Comfy Implementation would be awesome.

Phando avatar Jan 31 '24 18:01 Phando

Super excited to see this @threefoldo

Haven't maken any progress. The code is mixed with webui code, not sure how to seperate them.

Hi @threefoldo , I created an WebUI extension sd-webui-rpg-diffusionmaster. Maybe you can find some inspirations from it.

zydxt avatar Feb 03 '24 14:02 zydxt

Why exactly was this issue closed its is not resolved. There is an webui extension available but no comfy UI implementation as far as https://ltdrdata.github.io/ listing.

Kademo15 avatar Feb 07 '24 23:02 Kademo15

Hello @threefoldo do you have news for us, I hope you didn't give up on this.

Kademo15 avatar Feb 17 '24 16:02 Kademo15

I'm trying to replicate how this works with my oobabooga nodes. I'm currently experimenting with ImpactPack's regional prompt nodes, so I can figure out how to tie them together.

EDIT: My current thinking is that there will be a node to provide a suitable input to the Oobabooga nodes, another to extract the values from the LLM response, and another to convert those to the regional prompts which can be fed into the regional prompt sampler. Feel free to speak up if any of that sounds like it won't work (or you have an alternative idea!)

Xyem avatar May 03 '24 23:05 Xyem

I'm trying to replicate how this works with my oobabooga nodes. I'm currently experimenting with ImpactPack's regional prompt nodes, so I can figure out how to tie them together.

EDIT: My current thinking is that there will be a node to provide a suitable input to the Oobabooga nodes, another to extract the values from the LLM response, and another to convert those to the regional prompts which can be fed into the regional prompt sampler. Feel free to speak up if any of that sounds like it won't work (or you have an alternative idea!)

If you want to apply a Regional Prompt, you might need an adapter node that generates a mask on a grid basis using a text prompt.

ltdrdata avatar May 04 '24 01:05 ltdrdata

If you want to apply a Regional Prompt, you might need an adapter node that generates a mask on a grid basis using a text prompt.

Yes, this would be one of the things that the node that converts the extracted values to regional prompts would do. It would be also generating the mask for the area the LLM defines for each region.

I have already got my Oobabooga nodes set up to get a response that I think it similar to what RPG-DiffusionMaster would be outputting (I can't check because don't have enough VRAM to run it). I'm currently getting to know the RegionalPrompt nodes from your Packs before I start coding these extra nodes to get them working together.

Xyem avatar May 04 '24 18:05 Xyem

+1 would love to use this in ComfyUI

andupotorac avatar Aug 12 '24 23:08 andupotorac

+1 would love to use this in ComfyUI

There is a custom node implemented by something called omost + densediffusion that have a similar approach to RPG-Diffusion. I recommend you refer to them.

https://github.com/huchenlei/ComfyUI_omost https://github.com/huchenlei/ComfyUI_densediffusion

ltdrdata avatar Aug 13 '24 15:08 ltdrdata

+1 would love to use this in ComfyUI

There is a custom node implemented by something called omost + densediffusion that have a similar approach to RPG-Diffusion. I recommend you refer to them.

https://github.com/huchenlei/ComfyUI_omost https://github.com/huchenlei/ComfyUI_densediffusion

It's not a similar approach at all. :)

andupotorac avatar Aug 13 '24 16:08 andupotorac