Skip to content

Update sharding for activations to include 'dp' axis. Validated on internal notebook, after enabling (2, 1, 4) for (dp, fsdp, tp), do see weights replicated on dp axis. #849

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.

Sign up for GitHub

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

Open
copybara-service wants to merge 1 commit into
base: main
Choose a base branch
from
+20 −15

Update sharding for activations to include 'dp' axis. Validated on in…

935c88d
Select commit
Loading
Failed to load commit list.
Open

Update sharding for activations to include 'dp' axis. Validated on internal notebook, after enabling (2, 1, 4) for (dp, fsdp, tp), do see weights replicated on dp axis. #849

Update sharding for activations to include 'dp' axis. Validated on in…
935c88d
Select commit
Loading
Failed to load commit list.
Google CLA / cla/google succeeded Dec 5, 2025 in 3s

✅ All contributors are covered under a CLA with Google

See https://cla.developers.google.com/ for more info about Google's Contributor License Agreement (CLA).

ℹ️ Googlers: Go here to view more details and manage scans for this pull request.

Details

The following contributors were found for this pull request:

✅ 935c88d PR Opener: @copybara-service[bot]
✅ 935c88d Author: @wang2yn84 <la*****ng​@google.com>

(Only the first commit for a unique contributor is listed.)

View more details on Google CLA