Skip to content
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

Refine XCCL build option #1443

Merged
merged 1 commit into from
Mar 11, 2025
Merged

Refine XCCL build option #1443

merged 1 commit into from
Mar 11, 2025

Conversation

guangyey
Copy link
Contributor

@guangyey guangyey commented Mar 10, 2025

Motivation

cherry pick from #1421
Prior to this PR, building PyTorch XPU with CCL distribution support required setting USE_XCCL=ON USE_C10D_XCCL=ON python setup.py install.
With this PR, it only requires USE_XCCL=ON python setup.py install, simplifying the process.

@guangyey guangyey requested a review from xytintel March 10, 2025 02:24
@guangyey guangyey force-pushed the guangyey/oneccl_fix branch from 2d4fd2c to 7d1e00d Compare March 11, 2025 03:53
@guangyey guangyey changed the title Fix when USE_C10D_XCCL define in pytorch will not correct update cache Refine XCCL build option Mar 11, 2025
@guangyey guangyey requested a review from EikanWang March 11, 2025 03:59
@guangyey
Copy link
Contributor Author

The failure is irrelevant.

Show Failed cases in op_regression
=========================================================================
test_torchvision_roi_ops.py::TestRoIAlign::test_autocast[rois_dtype1-x_dtype0-True-True]
test_torchvision_roi_ops.py::TestRoIAlign::test_autocast[rois_dtype1-x_dtype0-True-False]

@guangyey guangyey merged commit 9210fa2 into release/2.7 Mar 11, 2025
8 of 9 checks passed
@guangyey guangyey deleted the guangyey/oneccl_fix branch March 11, 2025 06:59
# Motivation
Prior to this PR, building PyTorch XPU with CCL distribution support
required setting `USE_XCCL=ON USE_C10D_XCCL=ON python setup.py install`.
With this PR, it only requires `USE_XCCL=ON python setup.py install`,
simplifying the process.

(cherry picked from commit d799bd3)
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.

2 participants