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

[Question] IPAdapterAttnProcessor2_0 in diffusers is Different from IPAttnProcessor2_0 in AP-adapter source #452

Open
MrWH123 opened this issue Dec 17, 2024 · 0 comments

Comments

@MrWH123
Copy link

MrWH123 commented Dec 17, 2024

hi ,@xiaohu2015

I get one question on the source in diffusers and in this repo.

Seams in diffusers, each image embedding tokens are processed one-by-one in attention module.

attention_processor.py
image
image
this operation means ip-attention is calculated on the first image token, then update the hidden_states. then the second one and so on.

Seams quite strange.

while in the original repo,
attention_processor.py
image
image
all image tokens are processed in one attention calculation op, that's the way I think correct.

So could you help to confirm if this is an issue or I made a mistake. Thanks so much

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

No branches or pull requests

1 participant