KEMBAR78
adjust to get CI test cases passed on XPU by kaixuanliu · Pull Request #11759 · huggingface/diffusers · GitHub
Skip to content

Conversation

@kaixuanliu
Copy link
Contributor

No description provided.

Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
@kaixuanliu
Copy link
Contributor Author

kaixuanliu commented Jun 19, 2025

W/ this PR, we can get following failed cases passed on XPU:

- pytest -rA tests/pipelines/pag/test_pag_kolors.py::KolorsPAGPipelineFastTests::test_float16_inference
- pytest -rA tests/pipelines/kandinsky2_2/test_kandinsky_controlnet.py::KandinskyV22ControlnetPipelineIntegrationTests::test_kandinsky_controlnet
- pytest -rA tests/quantization/gguf/test_gguf.py::SD35LargeGGUFSingleFileTests::test_pipeline_inference
- pytest -rA tests/pipelines/ledits_pp/test_ledits_pp_stable_diffusion.py::LEditsPPPipelineStableDiffusionSlowTests::test_ledits_pp_editing

@kaixuanliu
Copy link
Contributor Author

@a-r-r-o-w @DN6 , Hi, we are deprecating IPEX in our software stack. Hence there will be some precision variance for model output in test cases. pls help review, thx. @yao-matrix FYI.

@kaixuanliu
Copy link
Contributor Author

@bot /style

Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
@DN6
Copy link
Collaborator

DN6 commented Jun 23, 2025

@bot /style

@github-actions
Copy link
Contributor

Style fixes have been applied. View the workflow run here.

@kaixuanliu
Copy link
Contributor Author

Hi, @DN6 , Can you help merge?

@yao-matrix
Copy link
Contributor

@a-r-r-o-w , could you help merge it? Thx

@a-r-r-o-w
Copy link
Contributor

Started the tests, will merge as soon as CI is green. Thanks!

@a-r-r-o-w
Copy link
Contributor

Hmm, there seems to be some failures for tests that were already fixed in main. Let me try to merge and run again

@a-r-r-o-w
Copy link
Contributor

Failing test seems to be unrelated to changes in this PR

@a-r-r-o-w a-r-r-o-w merged commit dd28509 into huggingface:main Jun 25, 2025
24 of 25 checks passed
@kaixuanliu kaixuanliu deleted the xpu-precision-ut branch June 25, 2025 08:35
@sayakpaul
Copy link
Member

This seems to cause this failure on a CUDA device:
https://github.com/huggingface/diffusers/actions/runs/15890953880/job/44813455615?pr=11705#step:8:943

@kaixuanliu could you please take a look?

@kaixuanliu
Copy link
Contributor Author

kaixuanliu commented Jun 26, 2025

@sayakpaul Sure, will try to fix.

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