Conversation
|
Hi @Skuld7451! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at cla@meta.com. Thanks! |
Summary
Closes #343
Related work
Draft PR #112 addresses one part of this problem: it waits for the first quorum
before forward can run. This PR has the same ordering requirement when
init_sync=True; unlike #112, it preserves asynchronous startup wheninit_sync=False.The remaining changes in this PR are FSDP-specific and are not covered by #112:
If #112 lands first, this PR can rebase on it and remove the overlapping
first-quorum wait after the unconditional versus
init_sync-gated behavior isresolved. The FSDP recovery changes above remain necessary independently.
Dependencies
used by the recovery path.
not a prerequisite. If it lands first, this PR can rebase and remove the
overlapping wait after the startup policy is resolved.
Testing