Fixed SDXL single file loading to use the correct requested pipeline class by Mystfit · Pull Request #4142 · huggingface/diffusers (original) (raw)

What does this PR do?

Fixes #4108

When loading a SDXL model using from_single_file(), the returned pipeline is always an instance of StableDiffusionXLPipeline. This means that if you try to load a model into a StableDiffusionXLImg2ImgPipeline the returned pipeline is always StableDiffusionXLPipeline instead. This fix now respects the passed in pipeline_class argument to allow the user to pick which SDXL pipeline they would like to use with the model.

Todo

• Create/find/upload a tiny single file SDXL model similar to tiny-stable-diffusion-xl-pipe so that we can write a test that checks if from_single_file returns the correct pipeline without us downloading the entire SDXL model.

Before submitting

• This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
• Did you read the contributor guideline?
• Did you read our philosophy doc (important for complex PRs)?
• Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case.
• Did you make sure to update the documentation with your changes? Here are the
  documentation guidelines, and
  here are tips on formatting docstrings.
• Did you write any new necessary tests?

Who can review?

@patrickvonplaten

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.