Skip to content

adjust to get CI test cases passed on XPU#11759

Merged
a-r-r-o-w merged 8 commits into
huggingface:mainfrom
kaixuanliu:xpu-precision-ut
Jun 25, 2025
Merged

adjust to get CI test cases passed on XPU#11759
a-r-r-o-w merged 8 commits into
huggingface:mainfrom
kaixuanliu:xpu-precision-ut

Conversation

@kaixuanliu

Copy link
Copy Markdown
Contributor

No description provided.

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

kaixuanliu commented Jun 19, 2025

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
Contributor Author

@bot /style

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

DN6 commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator

@bot /style

@github-actions

Copy link
Copy Markdown
Contributor

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

@kaixuanliu

Copy link
Copy Markdown
Contributor Author

Hi, @DN6 , Can you help merge?

@yao-matrix

Copy link
Copy Markdown
Contributor

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

@a-r-r-o-w

Copy link
Copy Markdown
Contributor

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

@a-r-r-o-w

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

kaixuanliu commented Jun 26, 2025

Copy link
Copy Markdown
Contributor Author

@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