Skip to content

Conversation

@stephen-iezzi
Copy link
Contributor

What does this PR do?

Addresses part of #7064 by updating the IP adapter support for the SAG pipeline

Before submitting

Who can review?

@yiyixuxu

Copy link
Collaborator

@yiyixuxu yiyixuxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!
can we add IPAdapterTesterMixin to the fast test here

class StableDiffusionSAGPipelineFastTests(PipelineLatentTesterMixin, PipelineTesterMixin, unittest.TestCase):

similar to what's done in this PR #6862

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@stephen-iezzi
Copy link
Contributor Author

Added the tests!

@yiyixuxu yiyixuxu merged commit ca61287 into huggingface:main Mar 30, 2024
noskill pushed a commit to noskill/diffusers that referenced this pull request Apr 5, 2024
* fix ip adapter support

* Update sag pipelines tests, adjust sag pipeline to pass tests

---------

Co-authored-by: YiYi Xu <[email protected]>
sayakpaul pushed a commit that referenced this pull request Dec 23, 2024
* fix ip adapter support

* Update sag pipelines tests, adjust sag pipeline to pass tests

---------

Co-authored-by: YiYi Xu <[email protected]>
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.

3 participants