Skip to content

[FSDP][Replicate] added test cases to ensure replicate supports prefetching and sychronization after optimizer steps#160443

Closed
anshul-si wants to merge 3 commits intogh/anshul-si/22/basefrom
gh/anshul-si/22/head
Closed

[FSDP][Replicate] added test cases to ensure replicate supports prefetching and sychronization after optimizer steps#160443
anshul-si wants to merge 3 commits intogh/anshul-si/22/basefrom
gh/anshul-si/22/head

Conversation

@anshul-si
Copy link
Contributor

@anshul-si anshul-si commented Aug 12, 2025

Summary: In order to ensure that replicate acts as intended (a specialized version of hsdp) we need to make sure that it can pass the same tests that fully_shard can for training. The first test verifies Replicate's explicit prefetching functionality. The second verifies that Replicate correctly handles post-optimizer events.

Test Cases

  1. pytest test/distributed/_composable/test_replicate_training.py -k test_explicit_prefetching
  2. pytest test/distributed/_composable/test_replicate_training.py -k test_post_optim_event

Stack from ghstack (oldest at bottom):

cc @H-Huang @awgu @wanchaol @fegin @fduwjj @wz337 @wconstab @d4l3k @pragupta

…tching and sychronization after optimizer steps

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Aug 12, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/160443

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure

As of commit aa5dbb9 with merge base e6aa728 (image):

NEW FAILURE - The following job has failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

…ports prefetching and sychronization after optimizer steps"

**Summary:** In order to ensure that replicate acts as intended (a specialized version of hsdp) we need to make sure that it can pass the same tests that fully_shard can for training. The first test verifies Replicate's explicit prefetching functionality. The second verifies that Replicate correctly handles post-optimizer events.

**Test Cases**
1. pytest test/distributed/_composable/test_replicate_training.py -k test_explicit_prefetching
2. pytest test/distributed/_composable/test_replicate_training.py -k test_post_optim_event




cc H-Huang awgu wanchaol fegin fduwjj wz337 wconstab d4l3k pragupta

[ghstack-poisoned]
anshul-si added a commit to anshul-si/pytorch that referenced this pull request Aug 21, 2025
…tching and sychronization after optimizer steps

ghstack-source-id: 0883a0e
Pull Request resolved: pytorch#160443
anshul-si added a commit to anshul-si/pytorch that referenced this pull request Aug 26, 2025
…tching and sychronization after optimizer steps

ghstack-source-id: 0883a0e
Pull Request resolved: pytorch#160443
anshul-si added a commit to anshul-si/pytorch that referenced this pull request Aug 26, 2025
…tching and sychronization after optimizer steps

ghstack-source-id: 0883a0e
Pull Request resolved: pytorch#160443
…ports prefetching and sychronization after optimizer steps"

**Summary:** In order to ensure that replicate acts as intended (a specialized version of hsdp) we need to make sure that it can pass the same tests that fully_shard can for training. The first test verifies Replicate's explicit prefetching functionality. The second verifies that Replicate correctly handles post-optimizer events.

**Test Cases**
1. pytest test/distributed/_composable/test_replicate_training.py -k test_explicit_prefetching
2. pytest test/distributed/_composable/test_replicate_training.py -k test_post_optim_event




cc H-Huang awgu wanchaol fegin fduwjj wz337 wconstab d4l3k pragupta

[ghstack-poisoned]
anshul-si added a commit to anshul-si/pytorch that referenced this pull request Sep 2, 2025
…tching and sychronization after optimizer steps

ghstack-source-id: 0883a0e
Pull Request resolved: pytorch#160443
@anshul-si anshul-si closed this Sep 2, 2025
@github-actions github-actions bot deleted the gh/anshul-si/22/head branch October 3, 2025 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oncall: distributed Add this issue/PR to distributed oncall triage queue topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant