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

Change AllReduceSimplifier to handle trivial cross-partition all-reduces. #13919

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Jun 18, 2024

Change AllReduceSimplifier to handle trivial cross-partition all-reduces.

This CL ensures that AllReduceSimplifier can simplify trivial all-reduces (an all-reduce where each subgroup is formed of a single participant) that are not necessarily cross replica (for example a cross partition all-reduce). We only simplify non cross replica all-reduce when the module is SPMD.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#14073 from apivovarov:select_compare_algsimp 6fe68d7

@copybara-service copybara-service bot force-pushed the test_644267887 branch 4 times, most recently from dd61877 to d85384b Compare June 19, 2024 23:11
@copybara-service copybara-service bot changed the title Change AllReduceSimplifier to handle cross-partition all-reduces. Change AllReduceSimplifier to handle trivial cross-partition all-reduces. Jun 19, 2024
@copybara-service copybara-service bot force-pushed the test_644267887 branch 6 times, most recently from 3993b7f to 02e5efc Compare June 26, 2024 01:00
…ces.

This CL ensures that AllReduceSimplifier can simplify trivial all-reduces (an all-reduce where each subgroup is formed of a single participant) that are not necessarily cross replica (for example a cross partition all-reduce). We only simplify non cross replica all-reduce when the module is SPMD.

PiperOrigin-RevId: 646679054
@copybara-service copybara-service bot merged commit 3829a78 into main Jun 26, 2024
@copybara-service copybara-service bot deleted the test_644267887 branch June 26, 2024 01:51
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