Skip to content

Fix advanced_diffusion_training setup requirements (fixes #10142)#5

Open
srlynch1 wants to merge 1 commit into
mainfrom
e2e/diffusers-10142
Open

Fix advanced_diffusion_training setup requirements (fixes #10142)#5
srlynch1 wants to merge 1 commit into
mainfrom
e2e/diffusers-10142

Conversation

@srlynch1

Copy link
Copy Markdown
Owner

Summary

  • Add missing datasets, prodigyopt, and wandb to examples/advanced_diffusion_training/requirements.txt
  • Bump peft minimum version
  • Clarify that hf auth login and wandb login are only needed when using --push_to_hub and --report_to="wandb" respectively

Fixes huggingface#10142

Test plan

  • Verified requirements.txt lists all deps from issue repro
  • Verified README conditional auth note present
  • python utils/check_copies.py passes

Made with Cursor

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.

Requirements for advanced_diffusion_training incomplete

1 participant