Skip to content

Conversation

@MekkCyber
Copy link
Contributor

What does this PR do?

Fixes some issues with #42639

sywangyi and others added 5 commits December 9, 2025 15:37
Signed-off-by: Wang, Yi <yi.a.wang@intel.com>
Signed-off-by: Wang, Yi <yi.a.wang@intel.com>
Signed-off-by: Wang, Yi <yi.a.wang@intel.com>
@MekkCyber MekkCyber mentioned this pull request Dec 10, 2025
@HuggingFaceDocBuilderDev

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.

if not is_fbgemm_gpu_available():
if not is_torch_cuda_available() and not is_torch_xpu_available():
raise ImportError("Using fbgemm fp8 quantization requires a GPU or XPU")
if is_torch_cuda_available() and not is_kernels_available():
Copy link
Contributor

Choose a reason for hiding this comment

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

replace is_torch_cuda_available with is_torch_xpu_available?

@github-actions
Copy link
Contributor

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

run-slow: fbgemm_fp8

@MekkCyber MekkCyber requested a review from SunMarc December 11, 2025 07:30
Copy link
Member

@SunMarc SunMarc left a comment

Choose a reason for hiding this comment

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

let's go

@MekkCyber MekkCyber merged commit 86644be into main Dec 11, 2025
26 checks passed
@MekkCyber MekkCyber deleted the pr-42639 branch December 11, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants