Skip to content

[docs] Quicktour fixes#5211

Merged
patrickvonplaten merged 2 commits into
huggingface:mainfrom
stevhliu:update-quicktour
Sep 29, 2023
Merged

[docs] Quicktour fixes#5211
patrickvonplaten merged 2 commits into
huggingface:mainfrom
stevhliu:update-quicktour

Conversation

@stevhliu
Copy link
Copy Markdown
Member

Addresses questions raised in this Slack thread from @osanseviero

Should we showcase a lighter repo that does not contain so many different weights? Maybe ddpm?

Do you mean showcasing a repo that'll be faster to download or are you worried about the complexity of so many different weights in Stable Diffusion v1.5?

In the case of the latter, I think it may be ok to keep the current repo for continuity with the section above it and because it is the most popular model users that are probably interested in trying. I don't think they'll care too much about all the different weights in the repo since all they have to know right now is just how they can load the model using from_pretrained.

@osanseviero
Copy link
Copy Markdown
Contributor

There is a part in the guide in which we show users to git clone a repo, which is many GBs. I agree on the continuity point, thanks!

Copy link
Copy Markdown
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

Comment thread docs/source/en/quicktour.md Outdated
Comment thread src/diffusers/pipelines/pipeline_flax_utils.py
@patrickvonplaten
Copy link
Copy Markdown
Contributor

Addresses questions raised in this Slack thread from @osanseviero

Should we showcase a lighter repo that does not contain so many different weights? Maybe ddpm?

Do you mean showcasing a repo that'll be faster to download or are you worried about the complexity of so many different weights in Stable Diffusion v1.5?

In the case of the latter, I think it may be ok to keep the current repo for continuity with the section above it and because it is the most popular model users that are probably interested in trying. I don't think they'll care too much about all the different weights in the repo since all they have to know right now is just how they can load the model using from_pretrained.

Yes I don't think we need to change anything here

@patrickvonplaten patrickvonplaten merged commit 1d3120f into huggingface:main Sep 29, 2023
@stevhliu stevhliu deleted the update-quicktour branch September 29, 2023 16:41
chuzhdontcode pushed a commit to chuzhdontcode/diffusers that referenced this pull request Oct 4, 2023
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
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.

4 participants