Skip to content

[tests] QwenImage Layer tests#12858

Open
sayakpaul wants to merge 2 commits intomainfrom
qwenimage-layer-tests
Open

[tests] QwenImage Layer tests#12858
sayakpaul wants to merge 2 commits intomainfrom
qwenimage-layer-tests

Conversation

@sayakpaul
Copy link
Copy Markdown
Member

What does this PR do?

Adds tests for the newly added QwenImage layer model.

First, this PR fixes how we return the outputs; otherwise, we run into:

FAILED tests/pipelines/qwenimage/test_qwenimage_layered.py::QwenImageLayeredPipelineFastTests::test_callback_inputs - UnboundLocalError: local variable 'images' referenced before assignment

Second, we need to decide how to tackle this situation (which is fairly common across all failures):

FAILED tests/pipelines/qwenimage/test_qwenimage_layered.py::QwenImageLayeredPipelineFastTests::test_attention_slicing_forward_pass - TypeError: unsupported operand type(s) for -: 'list' and 'list'

Unlike other pipelines, we're always returning a list[Image.Image | np.ndarray | torch.Tensor]. One solution could be to apply np.stack() or torch.stack when output_type="np" or output_type="pt".

Cc: @yiyixuxu @naykun

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

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.

@sayakpaul
Copy link
Copy Markdown
Member Author

A gentle ping @yiyixuxu

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 1, 2026

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions bot added the stale Issues that haven't received updates label Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Issues that haven't received updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants