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

update the signature of from_single_file #7216

Merged
merged 6 commits into from
Mar 9, 2024
Merged

update the signature of from_single_file #7216

merged 6 commits into from
Mar 9, 2024

Conversation

yiyixuxu
Copy link
Collaborator

@yiyixuxu yiyixuxu commented Mar 5, 2024

updating the signature of from_single_file!
let me know if I missed anything

motivated from issues like this #7163

@yiyixuxu yiyixuxu requested review from DN6, sayakpaul and stevhliu March 5, 2024 00:52
@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.

Comment on lines 191 to 193
load_safety_checker (`bool`, *optional*, defaults to `False`):
Whether to load the safety checker model or not. By default, the safety checker is not loaded unless user pass
a `safety_checker` component in the `kwargs`.
Copy link
Member

Choose a reason for hiding this comment

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

Should we maybe include a note that this doesn't apply to models like SDXL or even better something like "This doesn't apply to all models. Such as: SDXL." WDYT?

Copy link
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.

Much needed. Ufff!

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.

Thanks so much for the update!

src/diffusers/loaders/single_file.py Outdated Show resolved Hide resolved
src/diffusers/loaders/single_file.py Outdated Show resolved Hide resolved
src/diffusers/loaders/single_file.py Outdated Show resolved Hide resolved
src/diffusers/loaders/single_file.py Outdated Show resolved Hide resolved
DN6 and others added 4 commits March 6, 2024 14:25
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Copy link
Collaborator

@DN6 DN6 left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏽

@yiyixuxu yiyixuxu merged commit e3cd6ca into main Mar 9, 2024
17 checks passed
@yiyixuxu yiyixuxu deleted the single-file-args branch March 9, 2024 03:51
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