Skip to content

Conversation

@anton-l
Copy link
Member

@anton-l anton-l commented Dec 12, 2022

In an effort to speed up our slow/push tests, this PR:

  • adds a @nightly decorator for tests that can be enabled with RUN_NIGHTLY=1
  • suggests num_inference_steps=3 for all @slow tests
  • moves end-to-end image reference-based @slow tests to @nightly
  • suggests num_inference_steps~50 for all @nightly tests

Nightly tests will be run daily at midnight by our CI.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Dec 12, 2022

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

@anton-l anton-l marked this pull request as ready for review December 16, 2022 08:34
@anton-l anton-l changed the title [WIP] Nightly integration tests Nightly integration tests Dec 16, 2022
@anton-l anton-l changed the title Nightly integration tests [WIP] Nightly integration tests Dec 16, 2022
@slow
@require_torch_gpu
class StableDiffusionImg2ImgPipelineIntegrationTests(unittest.TestCase):
class StableDiffusionImg2ImgPipelineSlowTests(unittest.TestCase):
Copy link
Member Author

@anton-l anton-l Dec 16, 2022

Choose a reason for hiding this comment

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

Img2Img is the most flaky, max_diff > 1e-4 even with 3 steps. If I won't find the cause today, will raise the tolerances

Copy link
Contributor

@patrickvonplaten patrickvonplaten Dec 16, 2022

Choose a reason for hiding this comment

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

Sounds good 1e-3 is totally fine. FYI in transformers we only have 5e-3 for some one-forward pass tests

Copy link
Contributor

@patrickvonplaten patrickvonplaten left a comment

Choose a reason for hiding this comment

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

Super cool! Looks all good to me :-)

Also definitely enough to start with stable diffusion for now and then maybe open some issue for things that should changed after :-)

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

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

Looking great! Just fixed a typo in the comment for RUN_NIGHTLY

@anton-l anton-l changed the title [WIP] Nightly integration tests Nightly integration tests Dec 16, 2022
@anton-l anton-l merged commit 086c7f9 into main Dec 16, 2022
@anton-l anton-l deleted the nightly-tests branch December 16, 2022 17:51
sliard pushed a commit to sliard/diffusers that referenced this pull request Dec 21, 2022
* [WIP] Nightly integration tests

* initial SD tests

* update SD slow tests

* style

* repaint

* ImageVariations

* style

* finish imgvar

* img2img tests

* debug

* inpaint 1.5

* inpaint legacy

* torch isn't happy about deterministic ops

* allclose -> max diff for shorter logs

* add SD2

* debug

* Update tests/pipelines/stable_diffusion_2/test_stable_diffusion.py

Co-authored-by: Patrick von Platen <[email protected]>

* Update tests/pipelines/stable_diffusion/test_stable_diffusion.py

Co-authored-by: Patrick von Platen <[email protected]>

* fix refs

* Update src/diffusers/utils/testing_utils.py

Co-authored-by: Pedro Cuenca <[email protected]>

* fix refs

* remove debug

Co-authored-by: Patrick von Platen <[email protected]>
Co-authored-by: Pedro Cuenca <[email protected]>
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
* [WIP] Nightly integration tests

* initial SD tests

* update SD slow tests

* style

* repaint

* ImageVariations

* style

* finish imgvar

* img2img tests

* debug

* inpaint 1.5

* inpaint legacy

* torch isn't happy about deterministic ops

* allclose -> max diff for shorter logs

* add SD2

* debug

* Update tests/pipelines/stable_diffusion_2/test_stable_diffusion.py

Co-authored-by: Patrick von Platen <[email protected]>

* Update tests/pipelines/stable_diffusion/test_stable_diffusion.py

Co-authored-by: Patrick von Platen <[email protected]>

* fix refs

* Update src/diffusers/utils/testing_utils.py

Co-authored-by: Pedro Cuenca <[email protected]>

* fix refs

* remove debug

Co-authored-by: Patrick von Platen <[email protected]>
Co-authored-by: Pedro Cuenca <[email protected]>
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