Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Adding FSDP Support to Training Library #213
Adding FSDP Support to Training Library #213
Changes from 42 commits
c6b14f4
cdca42c
1114539
2e33add
80e65d3
802f74f
d4851d1
ca10236
4a89949
5c3d96c
cb9fec7
32b7265
7a2ca02
b02fa03
b736c25
6d4bb46
64595d4
5f582c1
ca274f6
85ac691
2f96481
807d9f7
3907e38
0208e64
b5e6d38
d3010a7
d9b71c0
fa8c1ba
291acdb
eff3fef
edf7618
14f2b08
fdbe288
8fb86ca
ffb7fab
2b35ed1
065df3e
e472b4d
8f8dc8f
72abd67
7f7e9f2
723a85e
7cd6747
95eb2c0
dd47117
4cdfb8d
70ff83c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this need to be a factory? I think it can just be an assignment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm following the current convention set by DeepSpeedOptions in the file, so imo if we want to change this, we should make a follow-up PR that updates both of them