Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Flux ControlNet] Add support for de-distilled models with CFG #9635

Open
simbrams opened this issue Oct 10, 2024 · 8 comments · May be fixed by #9658
Open

[Flux ControlNet] Add support for de-distilled models with CFG #9635

simbrams opened this issue Oct 10, 2024 · 8 comments · May be fixed by #9658

Comments

@simbrams
Copy link

Flux with Inpaiting and ControlNet currently yields bad result with the base model.

To echo this comment, using de-distilled models could potentially help getting better outputs.

Currently the Flux ControlNet pipelines do not support models like https://huggingface.co/nyanko7/flux-dev-de-distill

@asomoza
Copy link
Member

asomoza commented Oct 10, 2024

Anyone wants to contribute with this? This should be a community pipeline in the meantime until we find if the new models are good and get popular, but this could produce better results than the current ones.

Also maybe someone can test this in ComfyUI and see if it makes a big difference.

@ighoshsubho
Copy link
Contributor

Anyone wants to contribute with this? This should be a community pipeline in the meantime until we find if the new models are good and get popular, but this could produce better results than the current ones.

Also maybe someone can test this in ComfyUI and see if it makes a big difference.

will take this up 😊

@charchit7
Copy link
Contributor

Hey, @ighoshsubho wanna collab on this?

@ighoshsubho
Copy link
Contributor

Hey, @ighoshsubho wanna collab on this?

sure

@taneeshamadhu18
Copy link

Anyone wants to contribute with this? This should be a community pipeline in the meantime until we find if the new models are good and get popular, but this could produce better results than the current ones.

Also maybe someone can test this in ComfyUI and see if it makes a big difference.
yes i would love to contribute on this

@ighoshsubho ighoshsubho linked a pull request Oct 12, 2024 that will close this issue
6 tasks
@a-r-r-o-w
Copy link
Member

Gentle ping @ighoshsubho

@ighoshsubho
Copy link
Contributor

@a-r-r-o-w I apologize, due to unexpected time constraints with my primary work commitments, I won't be able to give this the attention it deserves. Please feel free to reassign this to another contributor who can dedicate the proper time and effort to implement this feature.

@KarthikSundar2002
Copy link

I will give this issue a look, and I will start with testing this out in ComfyUI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants