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

fix bugs under kSgdSparseCpuTraining mode #100

Merged
merged 1 commit into from
Sep 23, 2016
Merged

fix bugs under kSgdSparseCpuTraining mode #100

merged 1 commit into from
Sep 23, 2016

Conversation

Zrachel
Copy link
Contributor

@Zrachel Zrachel commented Sep 20, 2016

  • Local training with "sparse_update = True" parameter triggers kSgdSparseCpuTraining mode, fix bugs under it.

@Zrachel Zrachel assigned Zrachel, qingqing01 and emailweixu and unassigned Zrachel Sep 20, 2016
@@ -141,7 +141,7 @@ void SgdThreadUpdater::traverse(GetTraverseCallback getTraverseCallback) {
} else if (hasCpuPara) {
getGlobalSyncThreadPool()->exec(cpuTraverse);
} else if (hasGpuPara) {
cpuTraverse(0, 0);
getGlobalSyncThreadPool()->exec(gpuTraverse);
Copy link
Collaborator

Choose a reason for hiding this comment

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

should be just "gpuTraverse(0,0)", otherwise line131-133 might be executed multiple times.

Local training with "sparse_update = True" parameter triggers kSgdSparseCpuTraining mode, fix bugs under it.
@Zrachel
Copy link
Contributor Author

Zrachel commented Sep 23, 2016

I've correct the issue above within the same commit, Please check it.

@emailweixu emailweixu merged commit 77ed98d into master Sep 23, 2016
@emailweixu emailweixu deleted the fix_bugs branch September 23, 2016 16:30
zhhsplendid pushed a commit to zhhsplendid/Paddle that referenced this pull request Sep 25, 2019
thisjiang pushed a commit to thisjiang/Paddle that referenced this pull request Oct 28, 2021
…ions. (PaddlePaddle#100)

* add arithmatic expr cmp

* add tests to cas

* add Mod support to CAS
wangxicoding pushed a commit to wangxicoding/Paddle that referenced this pull request Dec 9, 2021
* Fixed encoding bugs.

* modify docs

* Fixed bugs.
zhoutianzi666 pushed a commit to zhoutianzi666/Paddle that referenced this pull request May 23, 2022
update x86_linux_demo's model to 2.0
qizhaoaoe pushed a commit to qizhaoaoe/Paddle that referenced this pull request Mar 3, 2023
laipaang pushed a commit to laipaang/Paddle that referenced this pull request Dec 11, 2023
lizexu123 pushed a commit to lizexu123/Paddle that referenced this pull request Feb 23, 2024
jack603047588 pushed a commit to jiaoxuewu/PaddleBox that referenced this pull request Oct 29, 2024
WAYKEN-TSE pushed a commit to WAYKEN-TSE/Paddle that referenced this pull request Dec 6, 2024
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.

3 participants