Skip to content
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

dump to cpu #30750

Merged
merged 4 commits into from
Feb 1, 2021
Merged

dump to cpu #30750

merged 4 commits into from
Feb 1, 2021

Conversation

Thunderbrook
Copy link
Contributor

PR types

New features

PR changes

Others

Describe

支持dump gpu table到cpu

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Copy link
Contributor

@danleifeng danleifeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

}

// template <typename KeyType, typename ValType, typename GradType>
// void HeterComm<KeyType, ValType, GradType>::dump_to_cpu(int index) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

后续可以把注释删掉

@@ -72,6 +73,10 @@ class HeterComm {
return ((send_id / 4 != receive_id / 4) && (send_id + 4) % 8 != receive_id);
}

// void dump_to_cpu(int index);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

后续可以把注释删掉

@Thunderbrook Thunderbrook merged commit cb66c53 into PaddlePaddle:develop Feb 1, 2021
Thunderbrook added a commit to Thunderbrook/Paddle that referenced this pull request Mar 1, 2021
* dump to cpu

* format

* format

* format
fuyinno4 pushed a commit that referenced this pull request Mar 1, 2021
* solve build gpu task core (#30626)

* build gpu task core

* format

* dump to cpu (#30750)

* dump to cpu

* format

* format

* format

* support multi node in heterps (#31102)

* push multi node

* multi node

* MultiThread

* remove log

* solve bug in 30829

* optimizer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants