Skip to content

Conversation

@sywangyi
Copy link
Contributor

@sywangyi sywangyi commented Nov 3, 2025

…_model_generate_images

add xpu support in tests/models/janus/test_modeling_janus.py::JanusIntegrationTest::test_model_generate_images

@sywangyi
Copy link
Contributor Author

sywangyi commented Nov 3, 2025

@ydshieh please help review

@sywangyi
Copy link
Contributor Author

sywangyi commented Nov 3, 2025

The change in modeling is because: xpu set allow_is_causal_skip to True for prefill in static cache case for performance optimization in sdpa, as a result the attention_mask may not be contained in model_inputs.

@sywangyi
Copy link
Contributor Author

sywangyi commented Nov 6, 2025

Intel XPU: @IlyasMoutawwakil

@yao-matrix
Copy link
Contributor

@sywangyi , pls fix CI issues, thx.

Copy link
Collaborator

@ydshieh ydshieh left a comment

Choose a reason for hiding this comment

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

sound good, so it is kind of xpu specific, but general users might have use that flag too, so make sense.

Thank you. Please follow @yao-matrix to fix the failing tests. (about the modular files)

Signed-off-by: Wang, Yi A <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

[For maintainers] Suggested jobs to run (before merge)

run-slow: janus

@ydshieh ydshieh merged commit 370fc65 into huggingface:main Nov 8, 2025
15 checks passed
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.

3 participants