-
Notifications
You must be signed in to change notification settings - Fork 234
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
Add Llama7b FSDP test for torch.compile mode #818
Conversation
Is it a different test from the one that was added in #761 ? |
@regisss This attempts to add generic support for FSDP (similar to what you already have for DeepSpeed) such that going forward any model that runs with DeepSpeed can also be run with FSDP. Also I assume that tests in this file are "slow" tests and we can add longer tests here, we wanted to add models such as Llama-70B + FSDP which take longer time here. Test in #761 is a short test which is intended to run as part of Gaudi2 CI. Going forward if you feel there is repetition we can consider removing it. |
@pankd please add a test for Llama-70B also. |
15f2a97
to
54c5541
Compare
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
54c5541
to
91074d1
Compare
@regisss we are checking last few things, will let you know when it's done. |
91074d1
to
f828c71
Compare
@regisss this is ready for merge now. |
Adds a test for FSDP