Skip to content

Conversation

ehsan
Copy link
Contributor

@ehsan ehsan commented Feb 7, 2024

What does this PR do?

Fixes a regression in using the yaml module for reading the config file when converting a checkpoint to diffusers format. If no original config file is passed, original_config_file is set to a config yaml string. This PR updates the else branch to do the same thing there.

Fixes #6899

Before submitting

Who can review?

@sayakpaul

@sayakpaul sayakpaul requested a review from DN6 February 8, 2024 03:43
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.

Thanks!

Do you also have an example command that successfully runs with this PR?

@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.

@sayakpaul sayakpaul merged commit 17808a0 into huggingface:main Feb 8, 2024
@ehsan
Copy link
Contributor Author

ehsan commented Feb 8, 2024

Do you also have an example command that successfully runs with this PR?

For reference:

python3 /path/to/diffusers/scripts/convert_original_stable_diffusion_to_diffusers.py   --pipeline_class_name StableDiffusionXLPipeline  --checkpoint_path  /path/to/file.ckpt --original_config_file /path/to/config.yaml --dump_path /path/to/output

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.

Bug in convert_original_stable_diffusion_to_diffusers.py
3 participants