Skip to content

Conversation

@yao-matrix
Copy link
Contributor

  1. enable xpu for launcher -> validated
  2. expand cuda only ds uts to xpu -> all 3 passed
  3. expand profiler example to xpu -> validated

@SunMarc , pls help review, thx.

Signed-off-by: YAO Matrix <[email protected]>
To use this function absolutely zero calls to a CUDA device must be made in the notebook session before calling. If
any have been made, you will need to restart the notebook and make sure no cells use any CUDA capability.
To use this function absolutely zero calls to an accelerator must be made in the notebook session before calling.
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: I would maybe try to find better wording as accelerator - maybe device. Since the whole accelerate is built on top Accelerator class. If you had the same naming convention even before, we can keep it.

Copy link
Contributor Author

@yao-matrix yao-matrix Jun 26, 2025

Choose a reason for hiding this comment

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

@S1ro1 indeed, especially in context that the src code already uses Accelerator instance. Done, thx.

yao-matrix and others added 2 commits June 26, 2025 23:13
@yao-matrix
Copy link
Contributor Author

@S1ro1 @SunMarc , could you pls review again? Thx very much.

Copy link
Contributor

@S1ro1 S1ro1 left a comment

Choose a reason for hiding this comment

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

LGTM from me, let's wait for Marc, he should be back wednesday and can merge. Thank you!

@yao-matrix
Copy link
Contributor Author

@SunMarc , could you pls take a look? thx very much.

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.

Thanks !

@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.

@SunMarc
Copy link
Member

SunMarc commented Jul 7, 2025

@bot /style

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

Style bot fixed some files and pushed the changes.

@SunMarc SunMarc merged commit 1ac8643 into huggingface:main Jul 7, 2025
@yao-matrix yao-matrix deleted the xpu branch July 8, 2025 00:46
S1ro1 pushed a commit that referenced this pull request Jul 9, 2025
* 1. enable xpu for launcher 2. expand cuda only ds uts to xpu 3. expand profiler example to xpu

Signed-off-by: YAO Matrix <[email protected]>

* fix style

Signed-off-by: YAO Matrix <[email protected]>

* rename

Signed-off-by: YAO Matrix <[email protected]>

* Update profiler.py

* Apply style fixes

---------

Signed-off-by: YAO Matrix <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

4 participants