Skip to content

refactor(autoep-zero3): drop files already covered by #7938#19

Merged
nathon-lee merged 2 commits into
feat_autoEP_zero3from
fix_autoEP_zero3
May 22, 2026
Merged

refactor(autoep-zero3): drop files already covered by #7938#19
nathon-lee merged 2 commits into
feat_autoEP_zero3from
fix_autoEP_zero3

Conversation

@nathon-lee
Copy link
Copy Markdown
Owner

refactor(autoep-zero3): drop files already covered by deepspeedai#7938

PR deepspeedai#7938 (Add AutoEP) will be merged before this PR. Remove duplicate
content from this branch so the diff against master contains only the
ZeRO-3 integration delta after deepspeedai#7938 lands.

  • Reset to Add AutoEP deepspeedai/DeepSpeed#7938 version (no Z3-specific logic in this branch's copy):
    deepspeed/checkpoint/constants.py
    deepspeed/module_inject/auto_ep.py
    deepspeed/module_inject/auto_ep_config.py
    deepspeed/moe/{ep_count,ep_experts,ep_kernels,ep_repack,ep_router}.py
    deepspeed/runtime/{config,engine}.py

  • Reset auto_ep_layer.py to Add AutoEP deepspeedai/DeepSpeed#7938 and re-apply the Z3-only delta:
    tag expert params with _autoep_expert=True so ZeRO-3 skips DP
    partitioning for already-EP-partitioned experts.

  • Remove tests/unit/moe/test_autoep_smoke.py (byte-identical duplicate
    of test_autoep_zero3.py).

Remaining Z3-specific changes:
deepspeed/moe/utils.py
deepspeed/runtime/zero/{partition_parameters,stage3}.py
deepspeed/module_inject/auto_ep_layer.py (expert tagging)
docs/_tutorials/autoep.md
tests/pytest.ini (autoep marker)
tests/unit/moe/test_autoep_zero3.py

Signed-off-by: nathon-lee <leejianwoo@gmail.com>
scripts/check-license.py uses 'git grep -e ^# SPDX-License-Identifier: Apache-2.0$', which doesn't match compound expressions. Split 'Apache-2.0 AND BSD-3-Clause' into a primary SPDX line plus an 'Additional license' note; THIRD_PARTY_NOTICES.md still records the BSD-3-Clause portion.

Signed-off-by: nathon-lee <leejianwoo@gmail.com>
@nathon-lee nathon-lee merged commit 3d1de7c into feat_autoEP_zero3 May 22, 2026
5 of 8 checks passed
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