ComfyUI-fal-API icon indicating copy to clipboard operation
ComfyUI-fal-API copied to clipboard

flux img2img and inpaint?

Open jakechai opened this issue 1 year ago • 8 comments

Hi, is it possible to have the img2img version of flux/dev and flux-general and the Inpaint version of flux-general? Thanks.

jakechai avatar Oct 10 '24 08:10 jakechai

very useful nodes by @gokayfem . with some guidance on how to edit the nodes and add new ones, will send a PR

seshubonam avatar Oct 10 '24 08:10 seshubonam

@seshubonam Hello, I am planning to add an optional IMAGE parameter to flux general and flux dev, so we dont need to create 2 nodes for it. If someone try to img2img they just connect IMAGE if there is no IMAGE it should use Text to Image. (If you want to do this i can assign it to you). You can label them like Image for Img2Img and Mask for Inpainting for example.

You can send PRs by forking the repository then make changes on your fork. Commit and push your changes to your fork. After that go to your fork on github and it will give you an option to open a PR.

gokayfem avatar Oct 10 '24 13:10 gokayfem

will fork and update you on monday. 🙌🏻

seshubonam avatar Oct 10 '24 19:10 seshubonam

is it done? Will there be a denoise as well?

KINGLIFER avatar Oct 15 '24 21:10 KINGLIFER

I dont think he is doing it, i will do it myself then.

gokayfem avatar Oct 16 '24 02:10 gokayfem

oh sorry. caught up. sorry wont be able to do it.

seshubonam avatar Oct 16 '24 03:10 seshubonam

Please add that. Having that on Flux-dev would be awesome.

rasooul avatar Oct 18 '24 15:10 rasooul

I assume redux wont be added as well?

KINGLIFER avatar Dec 17 '24 00:12 KINGLIFER