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

PR #16938: Add NANOO FP8 support for collaborative communication unit tests #17265

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

copybara-service[bot]
Copy link

PR #16938: Add NANOO FP8 support for collaborative communication unit tests

Imported from GitHub PR #16938

This PR adds support for NANOO FP8 data format in the collaborative communication unit tests.

--
0fc74cc by Wen Chen [email protected]:

[AMD] Added NCCL support for fp8e4m3fnuz and fp8e5m2fnuz.

--
d247af5 by scxfjiang [email protected]:

refactor tests for collective comm ops

--
6f8c418 by scxfjiang [email protected]:

rafactor collective comm e2e tests

--
8ecb6ec by scxfjiang [email protected]:

update: replace str

--
338d3af by scxfjiang [email protected]:

get rid of macros

Merging this change closes #16938

FUTURE_COPYBARA_INTEGRATE_REVIEW=#16938 from ROCm:ci_dev_rccl_nanoo_fp8 338d3af

@copybara-service copybara-service bot force-pushed the test_675635116 branch 5 times, most recently from b209288 to c4f1a31 Compare September 18, 2024 09:29
… tests

Imported from GitHub PR #16938

This PR adds support for NANOO FP8 data format in the collaborative communication unit tests.
- For the context on OCP FP8 and NANOO FP8, please refer to this comment:
google/flax#3993 (comment)
- The unit tests in this PR are similar to GEMM unit test introduced in the following PR to be able to deal with both OCP and NANOO fp8 formats:
#10488
Copybara import of the project:

--
0fc74cc by Wen Chen <[email protected]>:

[AMD] Added NCCL support for fp8e4m3fnuz and fp8e5m2fnuz.

--
d247af5 by scxfjiang <[email protected]>:

refactor tests for collective comm ops

--
6f8c418 by scxfjiang <[email protected]>:

rafactor collective comm e2e tests

--
8ecb6ec by scxfjiang <[email protected]>:

update: replace str

--
338d3af by scxfjiang <[email protected]>:

get rid of macros

Merging this change closes #16938

FUTURE_COPYBARA_INTEGRATE_REVIEW=#16938 from ROCm:ci_dev_rccl_nanoo_fp8 338d3af
PiperOrigin-RevId: 675635116
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.

1 participant