-
Notifications
You must be signed in to change notification settings - Fork 6k
Test sigmas
in PipelineTesterMixin
#10101
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
base: main
Are you sure you want to change the base?
Conversation
Thanks @SahilCarterr! I think we can add this to the
diffusers/tests/pipelines/test_pipelines_common.py Lines 1700 to 1719 in 63b631f
cc @sayakpaul @DN6 WDYT? We currently test this for AYS schedule on
|
Indeed! Perhaps a better way to cover scheduler compatibility for pipelines would be to define a known scheduler support matrix somewhere at the top of the |
@SahilCarterr would you like to move this test into |
@SahilCarterr Thanks for moving the test! Looks like some changes to example notebooks got added to this PR, could we remove that? |
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. |
Just removed it. Notebooks were added due to |
Looks like this has revealed issues with some pipelines, I've found similar issues adding a test for |
Thank you for the update and for looking into these issues and further Continuing this PR ! I appreciate your efforts in working through them. @hlky |
sigmas
in PipelineTesterMixin
What does this PR do?
Added sigmas test for FluxPipelines
Continuation of Issues #10071 and #10081
Adding Test for PR #10081
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed.
@yiyixuxu