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

Add support for newer versions of diffusers #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AndPuQing
Copy link

According to the commit huggingface/diffusers@e828232#diff-852d5806e41c100ee99c68c9e1059bfad9c0177e6f4416f0faf5a2f2a6a64067 CrossAttention has renamed to Attention

And some forward parameters have changed.

Related: #2

Note: This PR is provided as a code snippet for reference and is not intended to be merged

@WindVChen WindVChen marked this pull request as ready for review January 17, 2024 15:16
@WindVChen
Copy link
Owner

Hi @AndPuQing,

Thank you for submitting this excellent PR! I've thoroughly tested the code, and it seamlessly integrates with the latest diffusers (0.25.0).

However, during my testing, I observed a significant disparity between the generated adversarial examples before and after, as illustrated below (left represents the original, while the right depicts the adapted version). Could you provide any insight into this observed difference?

Your cooperation is greatly appreciated.

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

Successfully merging this pull request may close these issues.

None yet

2 participants