-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[Tests] add test suite for SD3 DreamBooth #8650
Conversation
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. |
Will add more tests for prior preservation too in a separate PR once I fix it (reference: #8589). |
Really great idea!. Actually it helps a lot. I wanted to test it and all the tests fails for me and I know why, so I'll open a PR to fix it. |
@asomoza which tests are you referring to? |
Tested the lora ones: pytest examples/dreambooth/test_dreambooth_lora_sd3.py
============================================================ test session starts =============================================================
platform linux -- Python 3.12.3, pytest-8.2.2, pluggy-1.5.0
configfile: pyproject.toml
plugins: requests-mock-1.10.0, xdist-3.6.1, timeout-2.3.1
collected 3 items
examples/dreambooth/test_dreambooth_lora_sd3.py FFF [100%]
========================================================== short test summary info ===========================================================
FAILED examples/dreambooth/test_dreambooth_lora_sd3.py::DreamBoothLoRASD3::test_dreambooth_lora_sd3 - test_examples_utils.SubprocessCallException: Command `accelerate launch --config_file /tmp/tmp2yatc1ok/default_config.yml examples/dreamb...
FAILED examples/dreambooth/test_dreambooth_lora_sd3.py::DreamBoothLoRASD3::test_dreambooth_lora_sd3_checkpointing_checkpoints_total_limit - test_examples_utils.SubprocessCallException: Command `accelerate launch --config_file /tmp/tmp2yatc1ok/default_config.yml examples/dreamb...
FAILED examples/dreambooth/test_dreambooth_lora_sd3.py::DreamBoothLoRASD3::test_dreambooth_lora_sd3_checkpointing_checkpoints_total_limit_removes_multiple_checkpoints - test_examples_utils.SubprocessCallException: Command `accelerate launch --config_file /tmp/tmp2yatc1ok/default_config.yml examples/dreamb...
============================================================= 3 failed in 27.89s ============================================================= After applying #8743: ============================================================ test session starts =============================================================
platform linux -- Python 3.12.3, pytest-8.2.2, pluggy-1.5.0
configfile: pyproject.toml
plugins: requests-mock-1.10.0, xdist-3.6.1, timeout-2.3.1
collected 3 items
examples/dreambooth/test_dreambooth_lora_sd3.py ... [100%]
============================================================= 3 passed in 40.13s ============================================================= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍🏽 Good to merge once #8743 is handled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Maybe it's worth to add a second test with the |
Thanks everyone for chiming in for this PR :) |
* add a test suite for SD3 DreamBooth * lora suite * style * add checkpointing tests for LoRA * add test to cover train_text_encoder.
What does this PR do?
SD3 training will be crucial in the coming days and we would want to ensure robustness for our canonical scripts. So, this PR adds test suites for the LoRA and non-LoRA DreamBooth scripts.