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 IPadapter] Support Xlabs IPadapter in diffusers #9403

Closed
chuck-ma opened this issue Sep 10, 2024 · 11 comments · Fixed by #10261
Closed

[Flux IPadapter] Support Xlabs IPadapter in diffusers #9403

chuck-ma opened this issue Sep 10, 2024 · 11 comments · Fixed by #10261

Comments

@chuck-ma
Copy link

It'd be great to have XLabs IPadapter supported in diffusers.

Code: https://github.com/XLabs-AI/x-flux/
Checkpoint: https://huggingface.co/XLabs-AI/flux-ip-adapter/tree/main

@sayakpaul

@sayakpaul
Copy link
Member

Feel free to start a PR if you're interested to contribute it. We can provide guidance along the way.

Cc: @fabiorigano too

@chuck-ma
Copy link
Author

Ok, I will have a try as long as I have time. Besides, I'm also curious if it will also work to make style transfer like we did in SDXL ipadapter.

@CONSTANT1386
Copy link

I have completed the training script and code for FLUX IP-Adapter based on diffusers, but I still encounter some issues in the final effect, Anyone can help me?

@asomoza
Copy link
Member

asomoza commented Sep 28, 2024

Hi, you probably will have better luck asking in the original code repository, we don't even have official support for inference here yet.

@TinyTigerPan
Copy link

I have completed the training script and code for FLUX IP-Adapter based on diffusers, but I still encounter some issues in the final effect, Anyone can help me?

Hi, I'm working on FLUX IP-Adapter training script based on x-flux. And I also meet some problem, maybe we can communicate together. Here is my email [email protected]

@max-fofanov
Copy link

Hello guys, I'm also interested in collaborating on this issue! My email is [email protected].

@vladmandic
Copy link
Contributor

with popularity of flux, this is becoming one of the top user requests - and hints on possible implementation?

@sayakpaul
Copy link
Member

Cc: @asomoza

Copy link

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions bot added the stale Issues that haven't received updates label Nov 19, 2024
@sayakpaul sayakpaul removed the stale Issues that haven't received updates label Nov 19, 2024
@tariq659891
Copy link

tariq659891 commented Dec 7, 2024

Hello @sayakpaul,

I came across your GitHub repo and had the chance to read your Word doc—amazing work! I'm a big fan! While exploring the repo, I looked for areas where I could contribute and stumbled upon this one. I started working on it last night.

So far, I haven’t found any existing implementation of the IP adapter in Diffusers, so I decided to give it a shot. I’ve implemented it, and while it’s not producing high-quality results yet, I can see the impact of the IP adapter.

I’m sharing the code here for anyone who’s interested in checking it out or continuing the work.

Additionally, I’ve included inference code that runs on less than 17GB of memory, with plans to optimize it further to go below 14GB.

https://github.com/tariq659891/Fluxforge-inference/tree/main

@sayakpaul
Copy link
Member

Hello @tariq659891 thanks for your kind words and also thanks for working on this.

I will let @asomoza comment on this further.

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.

9 participants