[ControlNet SDXL training] fixes in the training script by sayakpaul · Pull Request #4223 · huggingface/diffusers (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation2 Commits11 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Fixes #4206. Additionally

• adds a test for SDXL ControlNet training.
• adds licensing to the model card so that repos can have public visibility be default

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

sayakpaul changed the title[ControlNet SDXL training] specify the pipeline call arguments to avoid ambiguity [ControlNet SDXL training] fixes in the training script

Jul 24, 2023

sayakpaul marked this pull request as ready for review

July 24, 2023 14:38

Choose a reason for hiding this comment

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

Thanks!

orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request

Aug 1, 2023

…4223)

• fix: huggingface#4206

• add: sdxl controlnet training smoketest.

• remove unnecessary token inits.

• add: licensing to model card.

• include SDXL licensing in the model card and make public visibility default

• debugging

• debugging

• disable local file download.

• fix: training test.

• fix: ckpt prefix.

orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request

Aug 1, 2023

…4223)

• fix: huggingface#4206

• add: sdxl controlnet training smoketest.

• remove unnecessary token inits.

• add: licensing to model card.

• include SDXL licensing in the model card and make public visibility default

• debugging

• debugging

• disable local file download.

• fix: training test.

• fix: ckpt prefix.

orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request

Aug 1, 2023

…4223)

• fix: huggingface#4206

• add: sdxl controlnet training smoketest.

• remove unnecessary token inits.

• add: licensing to model card.

• include SDXL licensing in the model card and make public visibility default

• debugging

• debugging

• disable local file download.

• fix: training test.

• fix: ckpt prefix.

yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request

Dec 25, 2023

…4223)

• fix: huggingface#4206

• add: sdxl controlnet training smoketest.

• remove unnecessary token inits.

• add: licensing to model card.

• include SDXL licensing in the model card and make public visibility default

• debugging

• debugging

• disable local file download.

• fix: training test.

• fix: ckpt prefix.

AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request

Apr 26, 2024

…4223)

• fix: huggingface#4206

• add: sdxl controlnet training smoketest.

• remove unnecessary token inits.

• add: licensing to model card.

• include SDXL licensing in the model card and make public visibility default

• debugging

• debugging

• disable local file download.

• fix: training test.

• fix: ckpt prefix.