Skip to content

Conversation

@sayakpaul
Copy link
Member

What does this PR do?

  • Improve the LoRA doc to cut the clutter a bit and also include a mention of our support of TheLastBen LoRA.
  • Include fuse_lora() and unfuse_lora() APIs in the LoraLoaderMixin.

I think it'll be important to include these minor changes to prepare for the release.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Aug 30, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Contributor

@patrickvonplaten patrickvonplaten left a comment

Choose a reason for hiding this comment

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

Cool!

Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Niceee, thanks for adding these! 🔥


* [text_to_image/README_sdxl.md](https://github.com/huggingface/diffusers/blob/main/examples/text_to_image/README_sdxl.md)
* [dreambooth/README_sdxl.md](https://github.com/huggingface/diffusers/blob/main/examples/dreambooth/README_sdxl.md)
Here is an example:
Copy link
Member

Choose a reason for hiding this comment

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

Maybe also add some introductory text here instead of abruptly dropping the user straight into the code. Something like:

You can also load checkpoints from TheLastBen by specifying the model id and filename in the [`~diffusers.loaders.LoraLoaderMixin.load_lora_weights`] function:

@sayakpaul sayakpaul merged commit 19f3161 into main Aug 30, 2023
@sayakpaul sayakpaul deleted the lora-doc branch August 30, 2023 18:43
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* improve the LoRA doc.

* include fuse_lora and unfuse_lora

* Apply suggestions from code review

Co-authored-by: Steven Liu <[email protected]>

---------

Co-authored-by: Steven Liu <[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.

5 participants