-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Add auto completion module for auto parallel #34813
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
Merged
fuyinno4
merged 99 commits into
PaddlePaddle:develop
from
aoyulong:auto_parallel_completion
Aug 24, 2021
Merged
Changes from 96 commits
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
b985745
add auto_parallel dir
b79e749
mv to paddle.distributed
1671850
add shard_xx api
ec55a43
add distributed attrs for var
25abc00
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
bf24fb7
add ut, test=develop
8ea9363
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
9e4b3d8
add dist
e65f77e
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
8b95c1e
update
ccae6ae
update
d107751
update
f7e70ea
update
3111159
update
70cdb69
update, test=develop
9e5b0f0
update, test=develop
59936ef
update, test=develop
27ee413
update, test=develop
3a8ceef
update, test=develop
d11f317
update, test=develop
f5ef245
update, test=develop
7293b4f
update
1240edc
update
05455fb
update
3e1b3a0
update
8950c35
update
b94a9f2
update, test=develop
e121349
update, test=develop
fe51aa3
update
4563d42
update
192580d
Merge branch 'develop' into auto_parallel_basic
2e69980
delete unused proto
608dd3f
resotre op_desc
cb9b6bf
restore type_defs
8e6559e
update var_desc
00f5f4d
remove dimss_mapping for proto_pybind
1aa94da
update interface.py
97a446c
update framework.py
c586fc6
update
fc6cde9
update
9d1a664
add auto_parallel dir
5d1b472
mv to paddle.distributed
d1aabad
add shard_xx api
e6ba855
add distributed attrs for var
3bf613c
add ut, test=develop
8942a99
[WIP] Add the auto completion feature and related codes
aoyulong 6916cf2
[WIP] Improve the auto completion and related codes
aoyulong cafdd18
[WIP] Make the auto completion to support data-parallel
aoyulong 4d6dd52
[WIP] Make the completion support mp and dp+mp
aoyulong 3f05d09
[WIP] Refactor auto completion unit test for MLP
aoyulong 2c56e12
[WIP] Refactor the implementation of DistributedOperatorImpl
aoyulong a83e9cd
[WIP] Improve dims_mapping update rule and fix a bug
aoyulong 203ea14
[WIP] Support auto completion for one transformer decoder layer
aoyulong bbc2c39
[WIP] Add a minor change
aoyulong 2b6f992
[WIP] Fix a bug within the uint test
aoyulong 921c53d
Shard XShape tensor, add embedding completion and refactor code
aoyulong a03d503
Add the distributed_operators dir to setup.py.in
aoyulong 3770f13
Improve the completion process and add the unittest for gpt
aoyulong 967d0e7
fix process_mesh ut
cd1e390
fix process_mesh ut
f48ec91
update
b07affa
update, test=develop
f304b47
Add support for automatically completing distributed attrs of special…
aoyulong a00fe9e
update
da9fe30
update
3daecf2
update
5640879
fix doc sample codes, test=develop
05b0f82
improve coverage, test=develop
fe93d0e
add static_mode check, test=develop
033c541
Model the cluster for cost model and physical mapping
aoyulong 9856d47
update, test=develop
890c70c
add set_placement, test=develop
6291697
Add the check to make sure the candidate tensors' size is great than …
aoyulong 4b90b03
update doc, test=develop
c395b84
update doc, test=develop
8390e01
update doc, test=develop
f7d5631
update doc, test=develop
3a2666e
update, test=develop
fa98e39
Auto mark dist attrs annotated by user
aoyulong b5b8b9b
Merge branch 'PaddlePaddle:develop' into develop
aoyulong 70bc589
Merge branch 'PaddlePaddle:develop' into develop
aoyulong b9bd421
Merge PR#33804
aoyulong b59bc33
Merge branch 'PaddlePaddle:develop' into develop
aoyulong 773516b
update ndarray to nested list, test=develop
685504f
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
632eeac
Merge branch 'PaddlePaddle:develop' into develop
aoyulong 87abb4b
Merge branch 'pr_33804' into auto_parallel
aoyulong 7ac6299
update, test=develop
c724593
Add auto-completion module for auto-parallel (based on PR#33804)
aoyulong 63e66bc
Merge branch 'pr_33804' into auto_parallel
aoyulong 7087b1e
Merge branch 'PaddlePaddle:develop' into develop
aoyulong 1908acf
Merge branch 'develop' of https://github.com/aoyulong/Paddle into aut…
aoyulong 86ccd47
Remove unnecessary files
aoyulong 3f7dca2
Remove unrelated files for the auto completion pr
aoyulong ed02152
Update the unit test to improve the coverage
aoyulong 88e9e23
Modify codes based on reviews
aoyulong 63a6ec6
Minor changes for CI
aoyulong 6b77bc8
Improve some codes based on new comments
aoyulong 411507d
Merge branch 'auto_parallel_completion' of https://github.com/aoyulon…
aoyulong File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
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.
might name as dist_attr_id ? since by now it is used for determining the dist_attr identity only.
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.
The dist_attr_id is obsoleted in the new code because it cannot work well in different distributed contexts.
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 also have this question, maybe we can write in the comment that the
id_only be used for determining the dist_attr identity in auto_parallel now? avoid to make developers who read the code feel confused.