Skip to content

adjust to get CI test cases passed on XPU #11759

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jun 25, 2025

Conversation

kaixuanliu
Copy link
Contributor

No description provided.

@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

@DN6
Copy link
Collaborator

DN6 commented Jun 23, 2025

@bot /style

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
Member

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

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

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
Member

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

@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