add test and doc for QwenImageEdit Plus #12363
Conversation
|
|
||
| ## Multi-image reference with QwenImageEditPlusPipeline | ||
|
|
||
| TODO |
There was a problem hiding this comment.
Since it's the speciality of QwenImageEditPlusPipeline, I think we should have a section here.
There was a problem hiding this comment.
Do you want to include an example here for reference? 👀
|
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. |
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_num_images_per_prompt(): | ||
| super().test_num_images_per_prompt() | ||
|
|
||
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_inference_batch_consistent(): | ||
| super().test_inference_batch_consistent() | ||
|
|
||
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_inference_batch_single_identical(): | ||
| super().test_inference_batch_single_identical() |
There was a problem hiding this comment.
@naykun these are being xfailed for now because we have to fix how we handle a batch of batch of reference images. For the release, I think that's okay. But LMK.
There was a problem hiding this comment.
Yeah, let's fix this after the release~
There was a problem hiding this comment.
Thanks @sayakpaul , changes looks good for me. Is there a way for me to preview the doc?
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_num_images_per_prompt(): | ||
| super().test_num_images_per_prompt() | ||
|
|
||
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_inference_batch_consistent(): | ||
| super().test_inference_batch_consistent() | ||
|
|
||
| @pytest.mark.xfail(condition=True, reason="Batch of multiple images needs to be revisited", strict=True) | ||
| def test_inference_batch_single_identical(): | ||
| super().test_inference_batch_single_identical() |
There was a problem hiding this comment.
Yeah, let's fix this after the release~
|
|
||
| ## Multi-image reference with QwenImageEditPlusPipeline | ||
|
|
||
| TODO |
|
@naykun sorry I missed the message. You can preview the docs here: |
|
@linoytsaban thanks for the reference. I have pushed it. Will merge soon. |
What does this PR do?
Followup of #12357.
@naykun if you could give this a review.